Skip to content

fix: add LEARNINGS_SEARCH + LEARNINGS_LOG to 6 skills missing them#755

Open
Gonzih wants to merge 1 commit intogarrytan:mainfrom
Gonzih:fix/learnings-missing-skills
Open

fix: add LEARNINGS_SEARCH + LEARNINGS_LOG to 6 skills missing them#755
Gonzih wants to merge 1 commit intogarrytan:mainfrom
Gonzih:fix/learnings-missing-skills

Conversation

@Gonzih
Copy link
Copy Markdown

@Gonzih Gonzih commented Apr 1, 2026

Summary

autoplan, benchmark, canary, health, document-release, and land-and-deploy all perform complex multi-step analysis but were not wired into the gstack institutional memory system.

  • 7 skills (investigate, review, qa, cso, ship, retro, plan-*) already had {{LEARNINGS_SEARCH}} + {{LEARNINGS_LOG}}
  • 6 skills (the ones above) silently bypassed it entirely
  • This means: learnings from past benchmark/canary/deploy runs were never surfaced or persisted

Each skill gets:

  • {{LEARNINGS_SEARCH}} after initial setup — surfaces relevant learnings before starting analysis
  • {{LEARNINGS_LOG}} before Important Rules — persists new learnings at completion

Placement rationale

Skill LEARNINGS_SEARCH placement LEARNINGS_LOG placement
autoplan Before Phase 1 CEO review Before Important Rules
benchmark After Phase 1 setup, before page discovery Before Important Rules
canary After Phase 1 setup, before baseline capture Before Important Rules
health After Step 1 detect, before running tools Before Important Rules
document-release After Step 1 pre-flight, before per-file audit Before Important Rules
land-and-deploy After Step 1 pre-flight, before Step 1.5 dry-run Before Important Rules

Test plan

  • bun run gen:skill-docs regenerates all 6 files without errors
  • Static skill validation passes (550 tests)
  • Invoke /benchmark <url> and verify learnings are searched and persisted
  • Invoke /health and verify same

🤖 Generated with Claude Code

autoplan, benchmark, canary, health, document-release, and land-and-deploy
all perform complex multi-step analysis but were not surfacing prior
project learnings at the start or persisting new learnings at the end.

This means the gstack institutional memory system (LEARNINGS_SEARCH /
LEARNINGS_LOG) was silently bypassed for 6 of the 13 major skills.

Each skill gets:
- {{LEARNINGS_SEARCH}} after initial setup (surfaces relevant past learnings
  before starting the main analysis)
- {{LEARNINGS_LOG}} before Important Rules (persists learnings at completion)

Placement follows the same pattern as investigate, review, qa, cso, and
the other 7 skills that already had this integration.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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