Skip to content

docs(skills): add implement-batch skill for batch issue implementation - #51

Merged
s-annam merged 1 commit into
mainfrom
feat/implement-batch-skill
Jul 24, 2026
Merged

docs(skills): add implement-batch skill for batch issue implementation#51
s-annam merged 1 commit into
mainfrom
feat/implement-batch-skill

Conversation

@s-annam

@s-annam s-annam commented Jul 24, 2026

Copy link
Copy Markdown
Owner

Summary

Adds the implement-batch skill — a GitHub-only, self-contained batch driver for
okfmem. It implements an epic's sub-issues (or an explicit #,#,# list) onto
one feature branch as a single accumulated commit, running one isolated subagent
per issue (context-isolation is the mechanism, not an optimization), then a
bounded adversarial review pass, finalizing through /open-pr.

Adapted from the offlinecv sibling and customized to okfmem's facts.

No dedicated tracking issue — this is skill tooling. (Refs omitted.)

What changed vs the offlinecv original

  • Gates → pure-Python stdlib. Per-issue: scripts/check-leaks.py (hard) +
    affected pytest + advisory ruff check. Dropped the TS/React
    typecheck/build/fallow steps; the full OS-matrix verify is the PR gate.
  • Reviewer → ecc:python-reviewer (fallback ecc:code-reviewer, then a
    general-purpose subagent running /code-review).
  • Non-negotiable rule → public-repo leak rule (replaces fixture-PII):
    no private strings in tracked files; check-leaks.py must exit 0.
  • New blocking review classes specific to okfmem: confirmation-discipline
    rung (state-changing ops), and Windows/path invariants (pure-ASCII .ps1,
    path-by-shape, expected assertion values from the code's own normalizer).
  • Finalize → okfmem /open-pr (leak-gate preflight + single-commit collapse
    for linear history); s-annam/okfmem protected main, per-issue-keyword
    auto-close.

Retains: subagent-per-issue isolation, blocked_by topological ordering,
threaded handoff notes, per-stage self-reported provenance, the single Phase-1
human confirmation gate, and the marker-guarded ## Adversarial review PR-body
append.

Test plan

  • python3 scripts/check-leaks.py exits 0 (scanned 65 files, no leaks)
  • ruff check . reviewed (advisory; no Python changed — docs-only)
  • Skill file is well-formed markdown with valid frontmatter; discoverable as
    /implement-batch

Provenance

Skill authoring (adapt offlinecv → okfmem) via: Claude Opus 4.8 (1M context) — high effort
Verification: CI verify — pending

GitHub-only, self-contained batch driver customized for okfmem: implement an
epic's sub-issues (or an explicit list) onto one branch as a single accumulated
commit, one isolated subagent per issue, then a bounded adversarial review pass,
finalizing through /open-pr.

Adapted from the offlinecv sibling to okfmem's facts: pure-Python stdlib gates
(leak gate hard + affected pytest + advisory ruff, no typecheck/build/fallow),
ecc:python-reviewer, s-annam/okfmem protected main. Replaces the fixture-PII rule
with okfmem's non-negotiable public-repo leak rule, and adds confirmation-
discipline-rung and Windows/path invariants as blocking review classes.
@s-annam
s-annam merged commit 4264333 into main Jul 24, 2026
3 checks passed
@s-annam
s-annam deleted the feat/implement-batch-skill branch July 24, 2026 21:06
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.

1 participant