Skip to content

feature/Explain-opts-out-graph · L-260926-655aa3 - #88

Merged
lchoquel merged 1 commit into
devfrom
feature/Explain-opts-out-graph
Sep 26, 2026
Merged

lchoquel merged 1 commit into
devfrom
feature/Explain-opts-out-graph

Conversation

@lchoquel

@lchoquel lchoquel commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

The workshop's mthds_validate is about to write the method's flowchart, method-graph.html, beside files given by path unless the call passes graph_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 passes graph_page: false wherever 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, and docs/decisions.md records why /pipelex-organize and 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-explain and pipelex-catalog's "would this save?" check from leaving a method flowchart behind. The workshop's mthds_validate is about to write method-graph.html beside any files validated by path, and both skills write no files, so each now passes graph_page: false where the tool lists it. Closes L-260926-655aa3.

Notes

  • Both instructions are registered as guards in tests/unit/test_skill_guards.py.
  • docs/decisions.md records why pipelex-organize and the writing skills keep the page.
  • Changes apply to all three flavors (pipelex, pipelex-codex, pipelex-vibe) via their shared templates.

Written for commit aa106b1. Summary will update on new commits.

Review in cubic

…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
@lchoquel
lchoquel merged commit 18a5101 into dev Sep 26, 2026
8 checks passed
@lchoquel
lchoquel deleted the feature/Explain-opts-out-graph branch September 26, 2026 15:12
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