Skip to content

feature/Explain-shows-graph-page · L-260926-d89a39 - #89

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

lchoquel merged 1 commit into
devfrom
feature/Explain-shows-graph-page

Conversation

@lchoquel

@lchoquel lchoquel commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

Louis ruled that the workshop's method graph page is /pipelex-explain's default, so step 3 no longer passes graph_page: false and step 5 gives the page's path before the text flow, passing on the workshop's note about ignoring it and a page error in one line. The promise now says the skill writes nothing of its own and that the page is the one file it leaves, in the ratified wording; /pipelex-catalog's "would this save?" check keeps its opt-out, and the guards, the explain test, the docs and the Unreleased entry follow.

Closes L-260926-d89a39

🤖 Generated with Claude Code

https://claude.ai/code/session_01EJVC48w18XEdnvEc48jFAf


Summary by cubic

/pipelex-explain now keeps the method graph page the workshop writes during validation and points the explanation at it. Step 3 no longer passes graph_page: false; step 5 gives the page's path before the text flow, passes on the workshop's note about ignoring a new page under git, and gives a graph_page.error one line. The promise is retuned: the skill writes nothing of its own, and the page is the one file an explanation of a bundle on disk leaves.

  • pipelex-catalog's "would this save?" check still passes graph_page: false, since a question writes no file.
  • Skill guards, the explain test, docs/skills.md, docs/decisions.md and the Unreleased entry follow the new promise and pointer.

Closes L-260926-d89a39

Written for commit 52d8f8b. Summary will update on new commits.

Review in cubic

Louis ruled on 2026-09-26 that the flowchart page mthds_validate writes
beside path files is explain's default: step 3 no longer passes
graph_page: false, and step 5 gives the page's path before the text flow,
passes on the workshop's first-write note about ignoring it, and gives a
graph_page.error in one line. The promise now reads that the skill writes
nothing of its own and that the page is the one file it leaves, in the
description, the opening line, the guard and the stop row. The catalog's
"would this save?" check keeps its opt-out. Guards, the explain test,
docs/skills.md, docs/decisions.md and the Unreleased entry follow.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EJVC48w18XEdnvEc48jFAf
@lchoquel
lchoquel merged commit ce3b470 into dev Sep 26, 2026
8 checks passed
@lchoquel
lchoquel deleted the feature/Explain-shows-graph-page branch September 26, 2026 15:36
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