Skip to content

feat(skill): core SKILL.md integration (#7) - #14

Merged
andybaran merged 2 commits into
mainfrom
feat/7-skill-integration
Jul 16, 2026
Merged

feat(skill): core SKILL.md integration (#7)#14
andybaran merged 2 commits into
mainfrom
feat/7-skill-integration

Conversation

@andybaran

Copy link
Copy Markdown
Owner

Closes #7.

andybaran and others added 2 commits July 16, 2026 16:20
…, safety rails, worktrees

Closes #7.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…scratch file

Closes #7.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@andybaran

Copy link
Copy Markdown
Owner Author

🔬 Code review (code-review agent)

Verdict: APPROVED (after one revision round).

First pass raised 2 issues, both now fixed & re-verified:

  1. Importantreferences/security.md push-protection command switched from -f (stringified object → 422) to a proper nested JSON object via --input - heredoc, matching branch-protection.md. Now succeeds.
  2. Minor — SKILL.md Step 3.5 now instructs deleting .gitflow-review-package.txt before git add -A in the re-review loop, closing the accidental-commit gap.

Verified: scope limited to the 5 permitted files; frontmatter valid YAML (version: 1.1, allowed-tools, compatibility); documented commands match real gitflow.sh subcommands; CI check context (Validate skill) is real; all 18 intra-repo links resolve; evals.json parses with new scenarios (code-review verdict, CI gate, secret refusal, branch-protection consent). Non-negotiables, Step 0 offer-with-consent, Step 3.5 blocking review, Step 4 CI gate + revert rollback, worktrees, DoD, Quick reference all present.

Human orchestrator performing final squash-merge.

@andybaran
andybaran marked this pull request as ready for review July 16, 2026 20:28
@andybaran
andybaran merged commit f1102c6 into main Jul 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Core SKILL.md integration: verification gate, CI gating, safety rails, worktrees, code-review step

1 participant