Skip to content

feat: skill trigger discoverability + baseline convergence#87

Merged
benwu95 merged 4 commits into
mainfrom
benwu95/feat/converge-skill-triggers
Jul 12, 2026
Merged

feat: skill trigger discoverability + baseline convergence#87
benwu95 merged 4 commits into
mainfrom
benwu95/feat/converge-skill-triggers

Conversation

@benwu95

@benwu95 benwu95 commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Summary

Two stacked, skill-trigger–related changes, merged as one PR since both concern skill triggers.

1. emit-trigger-scaffold — make .prospec.yaml discoverable to onboarding agents

  • prospec agent triggers: emit a fill-missing skill_triggers localization scaffold (English baselines sourced from SKILL_DEFINITIONS) so onboarding agents translate values in place instead of grepping a deployed SKILL.md or reading the minified binary
  • prospec config example: print a complete, annotated .prospec.yaml reference covering every schema field (runs pre-init, in INIT_COMMANDS)
  • extract computeUnlocalizedSkills as the single-source fill-missing set, shared by the agent-sync hint and agent triggers
  • point the agent-sync hint and the quickstart/upgrade onboarding skills at prospec agent triggers
  • remove dead config schema fields (project.version, knowledge.files + KNOWLEDGE_FILE_TYPES, paths catchall); migrate deprecated .passthrough() → Zod 4 .loose()

2. converge-skill-triggers — tighten over-broad trigger baselines

  • converge 8 skills' triggers to prospec-specific, collision-free, ≥3-word sets; drop bare over-broad words that mis-fire on everyday dev talk (upgrade, I want to, setup, done, critical, feedback, clean up, migrate version, version bump)
  • fix prospec-plan's sub-3 gap with technical plan (not implementation plan, which would collide with implement)
  • machine-enforce REQ-AGNT-033's ≥3-triggers intent via a new skill-format contract assertion (shared-predicate mutation guard)
  • mirror the convergence into this repo's .prospec.yaml Chinese skill_triggers

Testing

  • Full suite: 2131 tests green; typecheck + lint clean
  • collision contract (EN baselines + .prospec.yaml Chinese triggers) → 0 violations; all 17 skills ≥3 triggers
  • config-example completeness contract (parses against ProspecConfigSchema; every schema field present)
  • both changes went through the full SDD flow (story → plan → tasks → implement → review → verify grade A → archive)

Notes

  • The two changes are stacked on one branch (4 commits) and their specs are graduated into agent-integration / project-setup feature specs.

benwu95 added 4 commits July 12, 2026 14:07
- add `prospec agent triggers`: emit a fill-missing skill_triggers localization scaffold (English baselines from SKILL_DEFINITIONS)
- add `prospec config example`: print a complete, annotated .prospec.yaml reference covering every schema field (INIT_COMMANDS, runs pre-init)
- extract computeUnlocalizedSkills as the single-source fill-missing set, shared by the agent-sync hint and `agent triggers`
- point the agent-sync hint and quickstart/upgrade onboarding skills at `prospec agent triggers`
- remove dead config schema fields (project.version, knowledge.files + KNOWLEDGE_FILE_TYPES, paths catchall); migrate .passthrough() to .loose()
- add contract tests (scaffold YAML round-trip, config-example completeness, onboarding refs) + US-3 backward-compat
- sync READMEs, factual counts, and affected-module knowledge
- graduate agent-integration: US-439 + REQ-AGNT-036/SERVICES-066/TESTS-052 (ADDED); REQ-AGNT-021/TEMPLATES-108/121 (MODIFIED, point to `prospec agent triggers`)
- graduate project-setup: US-018/019 + REQ-CLI-021/TYPES-062/TESTS-051 (ADDED)
- copy archive summary to specs/_archived-history/2026-07-12-emit-trigger-scaffold.md
- bump product.md; refresh raw-scan.md
- harvest lessons ledger (module-readme manual counts +1; generated-file module-stale gotcha)
- converge 8 skills' triggers to prospec-specific, collision-free, >= 3-word sets (new-story/plan/review/verify/archive/learn/quickstart/upgrade); drop bare over-broad words (upgrade, I want to, setup, done, critical, feedback, clean up, migrate version, version bump)
- fix prospec-plan's sub-3 gap with `technical plan` (not `implementation plan`, which would collide with `implement`)
- machine-enforce REQ-AGNT-033's >= 3-triggers intent via a skill-format contract assertion with a shared-predicate mutation guard
- mirror the convergence into .prospec.yaml Chinese skill_triggers (still collision-free)
- re-sync deployed SKILL.md frontmatter + AGENTS.md via agent sync
- correct a stale templates reference count (20 -> 19) in module-map/index.md carried over from emit-trigger-scaffold
- graduate agent-integration: REQ-TESTS-053 (ADDED, >= 3-triggers contract); REQ-AGNT-033 (MODIFIED, baselines converged to prospec-specific + >= 3 machine-enforced)
- copy archive summary to specs/_archived-history/2026-07-12-converge-skill-triggers.md
- refresh raw-scan.md
- harvest lessons ledger (generated-file module-stale +1; test/structural-false-green +1 -- the >= 3 mutation guard was near-tautological)
@github-actions

Copy link
Copy Markdown

Prospec Check

Prospec drift check (structural — deterministic, zero LLM)

  PASS  req-references
  PASS  file-paths
  PASS  import-direction
  WARN  knowledge-health
  SKIP  task-completion — source unavailable: .prospec/changes/ not found (not version-controlled)
  PASS  dangling-prefix
  PASS  feature-modules
  PASS  mcp-readme-counts
  SKIP  review-provenance — source unavailable: .prospec/changes/ not found (not version-controlled)
  SKIP  metadata-completeness — source unavailable: .prospec/changes/ not found (not version-controlled)
  PASS  knowledge-size

Findings:
  [warn] prospec/ai-knowledge/modules/lib/README.md
         stale knowledge: module "lib" source last commit 2026-07-12T14:07:17+08:00 is newer than README last commit 2026-07-09T13:11:15+08:00

Knowledge coverage: 6/6 modules documented, 1 stale

Checked 8/11 checks: 0 fail, 1 warn, 3 skipped (not counted as pass)
Semantic consistency: not-checked (run /prospec-review)
Report written: prospec-report.json

@github-actions

Copy link
Copy Markdown

Test Coverage

Metric Coverage Covered / Total
lines 96.23% 3197 / 3322
statements 95.71% 3510 / 3667
functions 95.73% 584 / 610
branches 93.21% 1950 / 2092

@benwu95 benwu95 merged commit a83919c into main Jul 12, 2026
4 checks passed
@benwu95 benwu95 deleted the benwu95/feat/converge-skill-triggers branch July 12, 2026 08:43
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