feature/Explain-opts-out-graph · L-260926-655aa3 - #88
Merged
Merged
Conversation
…d graph page The workshop's mthds_validate is about to write method-graph.html beside files given by path unless graph_page is false. pipelex-explain promises to write nothing yet validates by path, so it now passes graph_page: false wherever the tool lists that argument; pipelex-catalog's optional "would this save?" check does the same. Both sentences are registered guards, and docs/decisions.md records why organize and the writing skills keep the page. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EJVC48w18XEdnvEc48jFAf
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.
The workshop's
mthds_validateis about to write the method's flowchart,method-graph.html, beside files given by path unless the call passesgraph_page: false, so/pipelex-explain, which promises to write nothing but validates by path, would have left the page in the user's bundle on every explanation. Explain now passesgraph_page: falsewherever the tool lists that argument, and so does/pipelex-catalog's optional "would this save?" check, which is a question and writes no file; both sentences are registered guards, anddocs/decisions.mdrecords why/pipelex-organizeand the writing skills keep the page.Closes L-260926-655aa3
🤖 Generated with Claude Code
https://claude.ai/code/session_01EJVC48w18XEdnvEc48jFAf
Summary by cubic
Stops
pipelex-explainandpipelex-catalog's "would this save?" check from leaving a method flowchart behind. The workshop'smthds_validateis about to writemethod-graph.htmlbeside any files validated by path, and both skills write no files, so each now passesgraph_page: falsewhere the tool lists it. Closes L-260926-655aa3.Notes
tests/unit/test_skill_guards.py.docs/decisions.mdrecords whypipelex-organizeand the writing skills keep the page.pipelex,pipelex-codex,pipelex-vibe) via their shared templates.Written for commit aa106b1. Summary will update on new commits.