Skip to content

feat: address all 7 open issues — full sweep#15

Merged
jw409 merged 6 commits intomainfrom
feat/issue-sweep-2026-03-31
Mar 31, 2026
Merged

feat: address all 7 open issues — full sweep#15
jw409 merged 6 commits intomainfrom
feat/issue-sweep-2026-03-31

Conversation

@jw409
Copy link
Copy Markdown
Owner

@jw409 jw409 commented Mar 31, 2026

Summary

Addresses all 7 open issues in a single sweep: 50 files changed, +1066/-14 lines.

Issues Addressed

Issue Title Type Status
#10 push_files not atomic bug fix Error context + 2 tests
#14 beads:create auto-enrich skill enhancement Batch mode workflow
#6 Parameterize all agents/skills enhancement 22 agents + 24 skills done
#8 Gemini CLI porting docs + compat Tool mapping guide
#3 GSD upstream evolution strategy Fork versioning + merge plan
#2 Sequential thinking subagents feature 2 new hint types + test
#9 Wargame extensibility design Simulation client interface

Rust Changes (tested)

  • kp-github: push_files step-context errors, orphan tracking, 20/20 tests pass
  • kp-sequential-thinking: subagent_spawn_available + subagent_orchestration hints, 20/20 tests pass

New Documents

  • docs/gemini-compatibility.md — Full tool name mapping + compat guide
  • docs/gsd-upstream-strategy.md — Selective merge plan for v1.27-v1.30
  • docs/simulation-extensibility.md — JSON interface for simulation clients

Test plan

  • kp-github: cargo test tools::files — 20 passed
  • kp-sequential-thinking: cargo test — 20 passed
  • Verify parameterized agent/skill files parse correctly (frontmatter intact)
  • Spot-check Gemini compatibility claims against actual Gemini CLI

jw409 added 6 commits March 31, 2026 15:19
47 files changed, +739/-14 lines
- test_subagent_orchestration_hint_on_three_branches (3 branches → hint fires)
- test_subagent_orchestration_suppressed_during_merge (merge → no hint)
- test_push_files_commit_creation_fails_reports_tree_plus_blobs (step 5)

22/22 seqthink tests, 21/21 push_files tests passing
- push_files: replace Vec<String> blob tracker with usize counter (reviewer issue 2)
- push_files: add orphan count to step 3 partial failure message (consistency)
- seqthink: deduplicate merge_available vs subagent_orchestration hints
  merge_available fires at exactly 2 branches, orchestration owns 3+ (reviewer issue 1/4)
Edit→replace (not edit_file), WebSearch→google_web_search,
WebFetch→web_fetch, Grep→search_file_content, TodoWrite→write_todos
- Rename test_push_files_atomic_* → test_push_files_* (no longer claims atomicity)
- Fix seqthink profile claims: both Gemini/Claude use depth 2, not 2 vs 3
- GSD strategy: add snapshot date, fix semver notation (- not +)
- Simulation doc: clarify var/archetypes/ is convention, not existing dir
@jw409 jw409 force-pushed the feat/issue-sweep-2026-03-31 branch from aee45d9 to a8ae59f Compare March 31, 2026 22:20
@jw409 jw409 merged commit 2472963 into main Mar 31, 2026
4 checks 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.

1 participant