fix: harden forge-global and always-on-guidance skills for compression resilience - #59
Open
jflowers wants to merge 5 commits into
Open
fix: harden forge-global and always-on-guidance skills for compression resilience#59jflowers wants to merge 5 commits into
jflowers wants to merge 5 commits into
Conversation
…n resilience Restructure two embedded skill files to improve survival under DCP context compression, addressing six specific vulnerabilities (unbound-force#51): - Replace parallel do/don't lists with decision table in forge-global - Inline TTL parameter into reservation step, add temporal ordering - Add Critical Safety section in always-on-guidance for safety rules - Move hivemind_find from last to first position in Tool Usage - Split Code Quality, Testing, Error Handling into sub-headed groups - Sync .opencode/skills/ copies with canonical embedded source No semantic changes to any rules -- structural reorganization only. Closes unbound-force#51 Assisted-by: claude-opus Generated with AI assistance (claude-opus)
sonupreetam
requested changes
Aug 6, 2026
sonupreetam
left a comment
sonupreetam
left a comment
Contributor
There was a problem hiding this comment.
Review Council
Ran the review council (6 Divisor agents). The restructuring patterns (decision table, temporal markers, priority positioning) are sound but needs structural tests and drift detection.
Cross-PR note: The Curator filed a blog opportunity issue for the prompt hardening methodology: unbound-force/website#198.
- always-on-guidance: Never -> NEVER force push to main (DR-002) - forge-global: add (5-minute auto-release) parenthetical to TTL Co-authored-by: claude-opus-4-6 <noreply@anthropic.com>
- TestAlwaysOnGuidance_StructuralHardening: verifies Critical Safety section positioning, RFC 2119 keywords, hivemind_find priority, and sub-header splits - TestForgeGlobal_StructuralHardening: verifies decision table format, temporal markers, inlined TTL, and no standalone TTL step - TestSkillFiles_DriftDetection: TC-007 drift detection comparing embedded files against .opencode/ scaffolded copies Co-authored-by: claude-opus-4-6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Restructures two embedded skill files to improve constraint survival under DCP context compression, addressing six specific vulnerabilities identified in #51.
forge-global: Replaces parallel "do/don't" lists with a decision table (Signal/Forge/Skip columns), inlines TTL parameter into the reservation step, and adds temporal ordering markers (FIRST/THEN/FINALLY) to protocol steps.
always-on-guidance: Adds a dedicated
## Critical Safetysection for safety-critical rules, moveshivemind_findfrom last to first position in Tool Usage, and splits Code Quality/Testing/Error Handling lists into sub-headed groups of 2-3 items each.All changes are structural reorganization -- no rules were added, removed, or weakened.
Closes #51
How to Test
Acceptance criteria from specs:
How to Demo
Inspect the restructured skill files:
Compare the decision table format in forge-global and the sub-headed groups in always-on-guidance against the original flat list structure.
Key Files Changed
internal/agentkit/content/skills/forge-global/SKILL.mdinternal/agentkit/content/skills/always-on-guidance/SKILL.md.opencode/skills/forge-global/SKILL.md.opencode/skills/always-on-guidance/SKILL.mdopenspec/changes/harden-skill-compression/*This PR was generated by /uf.finale (AI-assisted).