groombook/org
13
0
Fork 1
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Split devops and sdlc skills by scope; dedupe shared content #9

Merged
Chris Farhood merged 2 commits from claude/devops-sdlc-split into main 2026-05-28 01:18:56 +00:00
Conversation 0 Commits 2 Files Changed 2
+98 -53
Chris Farhood commented 2026-05-27 19:32:22 +00:00
Owner
Copy Link
Copy Source

Summary

  • devops/SKILL.md is now scoped to infrastructure (groombook/infra, single-branch main, Flux + OpenTofu controller, cluster topology, Flux image-automation policy).
  • sdlc/SKILL.md is scoped to application code (3-branch dev/uat/main, Phases 1-5, Stage 1 CI image build, application-tool policy).
  • Each skill cross-refs the other and defers to coding-standards / safety for cross-cutting rules instead of restating them.

Fixes in devops/SKILL.md

  • Rewrote the frontmatter description (was a verbatim copy of sdlc — referenced phases and dev/uat/prod that do not apply here).
  • Hoisted applies to groombook/infra to a top-level scope statement.
  • Renumbered the pipeline (was 1, 2, 3, 4, 4, 5, 4, 5, 5) and fixed --base dev to --base main in the tea example.
  • Closed an unterminated ** bold marker.
  • Removed Authentication framework, Stage 1 image build, and the never-tofu / never-kubectl-apply lines (cited from sdlc / safety).
  • Trimmed the tools list to infra-only operators and controllers.

Trims in sdlc/SKILL.md

  • Removed Infrastructure topology, IaC, Stage 2 GitOps detail, the Flux Image Automation DENIED policy, the never-tofu / never-kubectl-apply lines, and the External communication section (cited from devops / safety / coding-standards).
  • Trimmed the tools list to application-level dependency choices.
  • Added a pointer from Phase 5 into the devops pipeline.

Known ambiguity worth resolving separately

The sdlc Phase 5 PR to groombook/infra requires QA approval only, but the devops infra pipeline requires QA + CTO. Both flows operate on groombook/infra. This PR preserves both as written — flagging it for a policy call rather than inventing one.

Test plan

  • Skim devops/SKILL.md end-to-end; confirm no stray app-only content and the pipeline reads 1-8.
  • Skim sdlc/SKILL.md end-to-end; confirm no stray infra-only content and Phases 1-5 still make sense.
  • Confirm cross-refs resolve: every "see X" line points to a section that actually exists in skill X.
  • Decide the Phase 5 vs devops-pipeline QA/CTO question above.

cc @cpfarhood

## Summary - `devops/SKILL.md` is now scoped to **infrastructure** (`groombook/infra`, single-branch `main`, Flux + OpenTofu controller, cluster topology, Flux image-automation policy). - `sdlc/SKILL.md` is scoped to **application code** (3-branch `dev`/`uat`/`main`, Phases 1-5, Stage 1 CI image build, application-tool policy). - Each skill cross-refs the other and defers to `coding-standards` / `safety` for cross-cutting rules instead of restating them. ## Fixes in devops/SKILL.md - Rewrote the frontmatter description (was a verbatim copy of sdlc — referenced phases and dev/uat/prod that do not apply here). - Hoisted *applies to groombook/infra* to a top-level scope statement. - Renumbered the pipeline (was 1, 2, 3, 4, 4, 5, 4, 5, 5) and fixed --base dev to --base main in the tea example. - Closed an unterminated ** bold marker. - Removed Authentication framework, Stage 1 image build, and the never-tofu / never-kubectl-apply lines (cited from sdlc / safety). - Trimmed the tools list to infra-only operators and controllers. ## Trims in sdlc/SKILL.md - Removed Infrastructure topology, IaC, Stage 2 GitOps detail, the Flux Image Automation DENIED policy, the never-tofu / never-kubectl-apply lines, and the External communication section (cited from devops / safety / coding-standards). - Trimmed the tools list to application-level dependency choices. - Added a pointer from Phase 5 into the devops pipeline. ## Known ambiguity worth resolving separately The sdlc Phase 5 PR to groombook/infra requires **QA approval only**, but the devops infra pipeline requires **QA + CTO**. Both flows operate on groombook/infra. This PR preserves both as written — flagging it for a policy call rather than inventing one. ## Test plan - [ ] Skim devops/SKILL.md end-to-end; confirm no stray app-only content and the pipeline reads 1-8. - [ ] Skim sdlc/SKILL.md end-to-end; confirm no stray infra-only content and Phases 1-5 still make sense. - [ ] Confirm cross-refs resolve: every "see X" line points to a section that actually exists in skill X. - [ ] Decide the Phase 5 vs devops-pipeline QA/CTO question above. cc @cpfarhood
Chris Farhood added 1 commit 2026-05-27 19:32:22 +00:00
Split devops and sdlc skills by scope; dedupe shared content d6b13fa58d 
devops/SKILL.md is now the canonical home for infrastructure lifecycle
(groombook/infra, single-branch main, Flux + OpenTofu controller, cluster
topology). sdlc/SKILL.md is scoped to application code (3-branch dev/uat/main,
Phases 1-5, Stage 1 CI image build, app-tool policy). Each skill cross-refs
the other and defers to coding-standards/safety for cross-cutting rules
rather than restating them.

Fixes in devops/SKILL.md:
- Rewrote frontmatter description (was a copy of sdlc, referenced phases
  and dev/uat/prod that do not apply).
- Hoisted "applies to groombook/infra" to a top-level scope statement.
- Renumbered the pipeline (was 1,2,3,4,4,5,4,5,5) and fixed --base dev
  -> --base main in the tea example.
- Closed an unterminated bold marker.
- Removed Authentication framework, Stage 1 image build, and the
  "never tofu / never kubectl apply" lines (now cited from sdlc / safety).
- Trimmed the tools list to infra-only operators and controllers.

Trims in sdlc/SKILL.md:
- Removed Infrastructure topology, IaC, Stage 2 GitOps detail, the Flux
  Image Automation DENIED policy, the "never tofu / never kubectl apply"
  lines, and the External communication section (cited from devops /
  safety / coding-standards instead).
- Trimmed the tools list to application-level dependency choices.
- Added a pointer from Phase 5 into the devops pipeline.

cc @cpfarhood

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Chris Farhood added 1 commit 2026-05-28 01:16:12 +00:00
Collapse sdlc Phase 5 to a redirect into the devops pipeline 11fd2a4c34 
Phase 5 is an infra PR against groombook/infra, which means it is governed
by the devops pipeline. Spelling out a separate (QA-only) review flow here
both duplicates devops and contradicted its QA+CTO requirement. Replaced
the step list with a one-paragraph hand-off.

Resolves the policy ambiguity flagged in the PR description.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Chris Farhood merged commit 4df9637518 into main 2026-05-28 01:18:56 +00:00
Chris Farhood deleted branch claude/devops-sdlc-split 2026-05-28 01:18:56 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 question
Further information is requested
 wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ai-review (AI Review) gb_barkley (Barkley Trimsworth) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) gb_flea (Flea Flicker) flux (Flux CD) admin (Gitea Admin) gb_lint (Lint Roller) renovate (Mend Renovate) gb_pawla (Pawla Abdul) gb_scrubs (Scrubs McBarkley) gb_shedward (Shedward Scissorhands) gb_dogfather (The Dogfather)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: groombook/org#9
Delete Branch "claude/devops-sdlc-split"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?