Repository navigation
fix(skill): the data-plane tree draws the sprint under docs/agents/, not .aladdin/ - #44
Merged
Merged
Conversation
…not .aladdin/ SKILL.md's layout tree listed `sprintN/` below `<project>/.aladdin/` (run data), and live run 24's haiku supervisor put the whole sprint in `.aladdin/sprint1/` (#41 made the gates refuse it). The tree now nests sprintN/ under docs/agents/ and draws .aladdin/ as run data only. scripts/test/skill-layout.test.py checks the tree and holds SKILL.md to a 50 KB budget so #43's trim does not grow back. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015QxyVf3NgwsgRyYpBgDDTC
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
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.
Requested by Adeel · project thread
Before: SKILL.md's data-plane tree listed
sprintN/directly below<project>/.aladdin/, the git-ignored run-data dir. Live run 24's haiku supervisor put the whole sprint in.aladdin/sprint1/. #41 made the gates refuse that, but the playbook still drew it.After: the tree nests
sprintN/underdocs/agents/..aladdin/is drawn separately as run data only, never a sprint. A new test checks the tree and holds SKILL.md to a 50 KB budget, so #43's trim cannot grow back unnoticed.How: test-first with
scripts/test/skill-layout.test.py, which failed on master with "sprintN/ is drawn under '/.aladdin/'". It is wired intomake ci, and the README suite count is bumped.make ciis green locally.This thread had built a similar playbook trim to #43, using an
aloop guidecommand. In a live run it measured 13/13, 12m44s, $0.78. I dropped it because #43 landed first with the same sections moved, and kept only this fix, which #43 did not include.Generated by Claude Code