Skip to content

fix(skills): consolidate gstack .tmpl split-mirror + restore refs (card kn79swtr)#3

Merged
LexClaw merged 3 commits into
mainfrom
reid/consolidate-tmpl-split-kn79swtr
Jun 4, 2026
Merged

fix(skills): consolidate gstack .tmpl split-mirror + restore refs (card kn79swtr)#3
LexClaw merged 3 commits into
mainfrom
reid/consolidate-tmpl-split-kn79swtr

Conversation

@LexClaw

@LexClaw LexClaw commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Summary

Consolidates the three stranded card-related commits on lexclaw/main (PR #1's split-mirror + the restore-refs fix + the hostall-regen report finding) into one coherent deliverable, so a routine bun run gen:skill-docs cannot re-freeze the 6 split gstack skills over the 100k Hermes skill_manage size gate.

Protects card kn75753n's unfreeze: without the .tmpl mirror, the next regen clobbers the split and re-freezes ship, plan-ceo-review, office-hours, plan-design-review, plan-devex-review, design-review.

Acceptance criteria

  • AC1 (each .tmpl reflects split): PASS — all 6 .tmpl carry the validation-log pointer.
  • AC2 (<95k for the 6): 5/6 PASS. office-hours 96444 (under 100k freeze wall, over 95k soft target) — flagged, held at 96444 because remaining body is irreducible core workflow.
  • AC3 (regen does NOT re-freeze, 0 skills >=100k): PASS — 0 of the 6 targets >=100k after regen.
  • AC4 (no content lost, logs readable via references/): PASS — all 6 references/validation-log.md present + non-empty.

Reproducibility

bun run gen:skill-docs is a clean no-op on tracked SKILL.md (git status empty after regen). The .tmpl ARE the source of the under-100k output.

Upstream-Candidate trailer present? no (LexClaw-local skill-size fix)

Checklist

  • Base branch is THIS fork's default (main), not upstream.
  • Commits carry Upstream-Candidate: trailers.
  • No LexClaw-specific code in upstream-eligible hunks (all local-forever).

Report: tasks/reports/2026-06-04-tmpl-split-consolidate-kn79swtr.md

Lex added 3 commits June 4, 2026 01:51
…7c4vnb)

Routes the stranded 9b937ba-equivalent template split-mirror changes onto the LexClaw fork and documents the gstack-upgrade fork patch path so Hit Network upgrades use LexClaw/gstack instead of losing local-forever template edits to upstream garrytan/gstack.

Upstream-Candidate: no
…(card kn79swtr)

The .tmpl split-mirror (PR #1, commit 789d4ca) moved the validation-log
sections out of the 6 gstack SKILL.md bodies and added pointers to
references/validation-log.md, but did NOT carry the reference files
themselves. The pointers became orphans and the audit content was
effectively lost on the durable LexClaw/gstack fork (fresh installs /
gstack-upgrade would ship dead pointers).

Restores the 6 references/validation-log.md files from the original
local fix commit 9b937ba so acceptance garrytan#4 (validation logs readable
via references/, no content lost) holds:
  ship 7521, plan-ceo-review 8156, office-hours 2709,
  plan-design-review 17158, plan-devex-review 16178, design-review 1588

Verified: bun run gen:skill-docs is a no-op on tracked SKILL.md
(references are not gen inputs); all 6 target SKILL.md bodies stay
under the 100k Hermes freeze wall after regen.

Upstream-Candidate: no
…rd kn79swtr)

Consolidates the three stranded card-related commits on lexclaw/main into
one coherent deliverable so a routine bun run gen:skill-docs cannot
re-freeze the 6 split gstack skills over the 100k Hermes size gate:

- .tmpl split-mirror (from 789d4ca, PR #1): templates carry the
  validation-log pointer; canonical SKILL.md regenerates under 100k.
- restored 6 references/validation-log.md files (from 80d14cb): PR #1
  left orphan pointers to missing reference files; restored so no
  audit content is lost on the durable fork.

Verified: bun run gen:skill-docs is a clean no-op on tracked SKILL.md
(reference files are not gen inputs). Post-regen sizes: ship 91966,
plan-ceo-review 92178, office-hours 96444, plan-design-review 91412,
plan-devex-review 90598, design-review 94330. Zero of the 6 targets
>=100k. office-hours is 1444 over the 95k soft target but 3556 under
the 100k freeze wall; its remaining body is irreducible core workflow,
so forcing it under 95k would extract a runtime phase. Judgment: hold
at 96444, AC3 (the load-bearing gate) fully met.

Upstream-Candidate: no
@LexClaw

LexClaw commented Jun 4, 2026

Copy link
Copy Markdown
Owner Author

Reid verification (card kn79swtr), clean-worktree regen from this branch HEAD (942b3ab):

AC1 (split lives in .tmpl): PASS. All 6 .tmpl files carry a references/validation-log.md pointer; generated SKILL.md bodies shrank accordingly.
AC2 (6 skills <95k): 5/6 PASS. office-hours = 96,444 bytes (95k-100k band). It dropped from 118,380 (frozen) to 96,444, so the 100k freeze wall that threatened card kn75753n is cleared. The literal <95k target is not met for office-hours by ~1.4k; the freeze-protection intent IS met.
AC3 (0 skills >=100k after gen:skill-docs --host all): PASS for all 6 target skills across every host adapter copy. NOTE: gstack-spec (~109k) and gstack-plan-eng-review (~107k) remain over 100k across all hosts. These are OUT OF SCOPE for this card (not among the 6) and were over 100k on main before this work; they are a separate pre-existing freeze item, not a regression introduced here.
AC4 (validation logs readable, no data loss): PASS. All 6 references/validation-log.md exist (1.5k-17k bytes) and SURVIVE a full regen (this restores the data dropped by PR #1).
Regen reproducibility: PASS, 0 drift (committed output == fresh --host all regen).

Supersedes PR #1 (split-mirror, dropped refs) and PR #2 (restore refs). Recommend landing #3 and closing #1/#2.

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