Skip to content

docs: fix workflow-deep-dive repo mislabel and validation checklist gap #32

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
smartwatermelon merged 1 commit into from
Aug 3, 2026
Merged

docs: fix workflow-deep-dive repo mislabel and validation checklist gap #32

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions docs/WORKFLOW-DEEP-DIVE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,11 +57,11 @@ Here is the **complete chronological order** of what fires during a typical deve

4. GITHUB ACTIONS (remote, costs $$$)
├─ claude-code-review.yml → automated PR code review
│ Source: .github/workflows/claude-code-review.yml (this repo)
│ Source: .github/workflows/claude-code-review.yml (smartwatermelon/claude-wrapper)
│ Uses: anthropics/claude-code-action@v1
│
└─ claude.yml → @claude mention handler
Source: .github/workflows/claude.yml (this repo)
Source: .github/workflows/claude.yml (smartwatermelon/claude-wrapper)
Delegates to: smartwatermelon/github-workflows/.github/workflows/claude-assistant.yml@v1

5. gh pr merge <number>
Expand Down Expand Up @@ -307,7 +307,7 @@ FINDING source=<bot> file="<path>" line=<line> comment=<text>

## Layer 6: GitHub Actions (Remote CI/CD)

**Source repo**: `smartwatermelon/claude-wrapper` (this repo)
**Source repo**: `smartwatermelon/claude-wrapper`
**Cost**: ~$0.008/minute on `ubuntu-latest`

### claude-code-review.yml — Automated PR Code Review
Expand Down Expand Up @@ -483,7 +483,7 @@ Developer writes code in Claude Code
▼
┌─────────────────────────────────────────────┐
│ GitHub Actions ($$$) │
│ Source: .github/workflows/ (this repo) │
│ Source: .github/workflows/ (claude-wrapper)│
│ ┌─────────────────────────────────────┐ │
│ │ claude-code-review.yml │ │
│ │ → anthropics/claude-code-action@v1 │ │
Expand Down Expand Up @@ -592,7 +592,7 @@ Developer writes code in Claude Code
|------|-------|---------|
| `gh` | 69 | Non-interactive shell gh wrapper |

### Source: `.github/workflows/` (this repo)
### Source: `.github/workflows/` (smartwatermelon/claude-wrapper)

| File | Triggers | Uses |
|------|----------|------|
Expand Down
3 changes: 3 additions & 0 deletions docs/plans/2026-04-02-phase2-local-whole-codebase-review.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -221,6 +221,8 @@ concern) on its first real run.

Track these across all repos during the validation week:

_Note: the Reliability item referencing dotfiles#29 was added 2026-08-03, after the 2026-04-10 validation window closed — added for completeness/tracking, not as a live validation task._

**Coverage parity:**

- [ ] Compare local codebase review findings with Seer findings on same PRs
Expand All @@ -245,6 +247,7 @@ Track these across all repos during the validation week:
- [ ] Any crashes or unparseable verdicts from codebase review?
- [ ] Does pre-merge-review.sh still work after the refactor?
- [ ] Any issues with non-blocking issue creation (gh failures, pending-issues fallback)?
- [ ] Does post-commit hook correctly report commit hash? (dotfiles#29 fix)

**Decision point (2026-04-10):**

Expand Down
Loading