ldraney/claude-custom
1
0
Fork 0
Code Issues 32 Pull requests Projects Releases Packages Wiki Activity Actions

Phase 13b: update-docs sync_board step + Phase 12 QA nit batch #103

New issue
Closed
opened 2026-03-14 23:31:34 +00:00 by forgejo_admin · 0 comments
forgejo_admin commented 2026-03-14 23:31:34 +00:00
Contributor
Copy link

Lineage

plan-pal-e-agency → Phase 13 (Post-Merge Workflow Modernization) → Phase 13b

Repo

forgejo_admin/claude-custom

User Story

As Betty Sue
I want the /update-docs command to include board sync and all accumulated nits fixed
So that post-merge workflow is complete and Phase 12 tech debt is resolved

Context

Phase 13 modernizes the post-merge workflow for continuous kanban. 13a (doc alignment in pal-e-docs) is being done separately. This issue covers the code changes in claude-custom.

Two categories of work:

  1. update-docs command — add explicit sync_board step after board item move
  2. Phase 12 QA nit batch — 6 nits accumulated across PRs #100 and #102 that were deferred per "merge first, nits in follow-up"

File Targets

Files to modify:

  • commands/update-docs.md — Add a sync_board step (Step 6) after the board item move. Renumber subsequent steps. The step should call sync_board(board_slug) on the relevant project board to reconcile drift.
  • hooks/inject-subagent-context.sh — Replace generic profile slugs: when agent type is dev-backend, inject agent-dev-backend (not agent-dev). Same for dev-frontend→agent-dev-frontend, devops→agent-devops, frontend-qa→agent-frontend-qa, dev-qa→agent-dev-qa, devops-qa→agent-devops-qa. Each agent type gets its own pal-e-docs profile slug.
  • schemas/agent-spawn-requirements.json — Update dev-backend description from generic to "Backend development agent — Python, FastAPI, SQLAlchemy, SDKs, MCP servers"
  • agents/betty-sue.md — Update Related section: replace agent-dev with agent-dev-frontend, agent-dev-backend, agent-devops. Replace agent-qa with agent-frontend-qa, agent-dev-qa, agent-devops-qa.
  • hooks/check-domain-label.sh — Change line 27 from BRE alternation to ERE (grep -E or equivalent) to match repo convention.
  • skills/review-pr/SKILL.md — Step 3: add state: "all" and limit: 50 to the list_issues guidance. Step 6: add note about spawn gate pattern requirements (must include issue ref + plan slug).

Files NOT to touch:

  • agents/dev-qa.md, agents/devops-qa.md, agents/frontend-qa.md, agents/devops.md — already correct
  • settings.json — no changes needed

Acceptance Criteria

  • commands/update-docs.md has a sync_board step
  • inject-subagent-context.sh uses domain-specific profile slugs for all 9 agent types
  • agent-spawn-requirements.json dev-backend description is accurate
  • betty-sue.md Related section references new agent names
  • check-domain-label.sh uses ERE syntax consistently
  • review-pr/SKILL.md Step 3 handles pagination, Step 6 notes spawn gate

Test Expectations

  • Verify inject-subagent-context.sh case statement covers all agent types with correct slugs
  • Verify commands/update-docs.md step numbering is sequential (no gaps or duplicates)
  • Verify check-domain-label.sh grep syntax is valid ERE

Constraints

  • Single PR batching all 7 changes — they're all small, independent fixes
  • Don't change any agent behavior — these are all correctness/consistency fixes
  • Keep commands/update-docs.md aligned with the canonical skill-update-docs note in pal-e-docs (being updated separately in 13a)

Checklist

  • PR opened
  • Tests pass
  • No unrelated changes

Closes: n/a

Related

  • plan-pal-e-agency — parent plan
  • phase-pal-e-agency-13-post-merge-modernization — phase note
### Lineage `plan-pal-e-agency` → Phase 13 (Post-Merge Workflow Modernization) → Phase 13b ### Repo `forgejo_admin/claude-custom` ### User Story As Betty Sue I want the /update-docs command to include board sync and all accumulated nits fixed So that post-merge workflow is complete and Phase 12 tech debt is resolved ### Context Phase 13 modernizes the post-merge workflow for continuous kanban. 13a (doc alignment in pal-e-docs) is being done separately. This issue covers the code changes in claude-custom. Two categories of work: 1. **update-docs command** — add explicit `sync_board` step after board item move 2. **Phase 12 QA nit batch** — 6 nits accumulated across PRs #100 and #102 that were deferred per "merge first, nits in follow-up" ### File Targets Files to modify: - `commands/update-docs.md` — Add a sync_board step (Step 6) after the board item move. Renumber subsequent steps. The step should call `sync_board(board_slug)` on the relevant project board to reconcile drift. - `hooks/inject-subagent-context.sh` — Replace generic profile slugs: when agent type is `dev-backend`, inject `agent-dev-backend` (not `agent-dev`). Same for `dev-frontend`→`agent-dev-frontend`, `devops`→`agent-devops`, `frontend-qa`→`agent-frontend-qa`, `dev-qa`→`agent-dev-qa`, `devops-qa`→`agent-devops-qa`. Each agent type gets its own pal-e-docs profile slug. - `schemas/agent-spawn-requirements.json` — Update `dev-backend` description from generic to "Backend development agent — Python, FastAPI, SQLAlchemy, SDKs, MCP servers" - `agents/betty-sue.md` — Update Related section: replace `agent-dev` with `agent-dev-frontend`, `agent-dev-backend`, `agent-devops`. Replace `agent-qa` with `agent-frontend-qa`, `agent-dev-qa`, `agent-devops-qa`. - `hooks/check-domain-label.sh` — Change line 27 from BRE alternation to ERE (`grep -E` or equivalent) to match repo convention. - `skills/review-pr/SKILL.md` — Step 3: add `state: "all"` and `limit: 50` to the list_issues guidance. Step 6: add note about spawn gate pattern requirements (must include issue ref + plan slug). Files NOT to touch: - `agents/dev-qa.md`, `agents/devops-qa.md`, `agents/frontend-qa.md`, `agents/devops.md` — already correct - `settings.json` — no changes needed ### Acceptance Criteria - [ ] `commands/update-docs.md` has a sync_board step - [ ] `inject-subagent-context.sh` uses domain-specific profile slugs for all 9 agent types - [ ] `agent-spawn-requirements.json` dev-backend description is accurate - [ ] `betty-sue.md` Related section references new agent names - [ ] `check-domain-label.sh` uses ERE syntax consistently - [ ] `review-pr/SKILL.md` Step 3 handles pagination, Step 6 notes spawn gate ### Test Expectations - [ ] Verify inject-subagent-context.sh case statement covers all agent types with correct slugs - [ ] Verify commands/update-docs.md step numbering is sequential (no gaps or duplicates) - [ ] Verify check-domain-label.sh grep syntax is valid ERE ### Constraints - Single PR batching all 7 changes — they're all small, independent fixes - Don't change any agent behavior — these are all correctness/consistency fixes - Keep commands/update-docs.md aligned with the canonical skill-update-docs note in pal-e-docs (being updated separately in 13a) ### Checklist - [ ] PR opened - [ ] Tests pass - [ ] No unrelated changes Closes: n/a ### Related - `plan-pal-e-agency` — parent plan - `phase-pal-e-agency-13-post-merge-modernization` — phase note
forgejo_admin 2026-03-14 23:38:05 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
217-enhance-skill-review-ticket-verify-arch
86-epilogue-nits-hook-updates-dottie-config
76-add-sessionstart-hook-to-detect-missing
69-fix-block-docs-writes-sh-security-gap-ad
51-phase-3-agent-label-behavior-posttooluse
No results found.
Labels
Clear labels   
domain:backend

   
domain:devops

   
domain:frontend

   
status:approved

QA passed, awaiting merge approval

  
status:in-progress

Dev agent is actively working

  
status:needs-fix

QA found issues, back to dev

  
status:qa

PR submitted, awaiting QA review

  
type:bug

Bug fix

  
type:devops

Infrastructure/CI/config work

  
type:feature

New functionality

No labels
domain:backend
domain:devops
domain:frontend
status:approved
status:in-progress
status:needs-fix
status:qa
type:bug
type:devops
type:feature
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ldraney/claude-custom#103
No description provided.