Repository navigation
feature/Slide-proposal-topic · L-261107-75b313 - #25
Merged
Merged
Conversation
…s mockup The image step now reads the polished brief and draws its topic word for word as the deck's title, and the polish keeps the topic as typed. The main pipe returns DesignProposals (the polished brief and one theme-and-mockup pair per proposal) instead of Html; the report moves to the exported generate_design_proposals_report, one card per proposal. The unused Slide and SlidePrompt concepts are removed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019TF9N3ixWPEiwAUJDxxxfd
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.
TL;DR
generate_design_proposals_from_rough_briefreturnsDesignProposalsinstead ofHtml; the HTML page is nowgenerate_design_proposals_report.Behaviour changes
topicword for word, and the other three slides carry placeholder content fitting the topic, the audience and the goal.DesignProposals: the polishedbriefand a list ofproposals, each athemewith itsmockup.Breaking changes
generate_design_proposals_from_rough_briefthat read theHtmloutput namesgenerate_design_proposals_reportinstead, which takes the samebriefand returns the page, now one card per proposal with its mockup beside its theme. A caller that wants the old pipe exactly stays at@v0.3.0. The next release is a minor,v0.4.0.Changes
methods/slide_designer/slide_designer.mthdsrender_visual_proposaltakespolished_briefbesidetheme, and its prompt names$polished_brief.topicas the title to draw word for word, keeping the four-slide mosaic.DesignProposalandDesignProposalsconcepts; adraft_proposalsub-sequence (draw, then pair through aPipeComposeconstruct) batched over the themes, thenassemble_proposals.generate_design_proposals_reportsequence;compose_proposals_reportreadsDesignProposalsand compares the original and polished briefs field by field.SlideandSlidePromptconcepts are removed, and the domain and pipe descriptions say what the method now does.methods/slide_designer/METHODS.tomlgenerate_design_proposals_reportbeside the main pipe; the description says what the result pairs.How it was verified
plxt fmt --checkandplxt linton the bundle and the manifest: clean.pipelex validate bundle methods/slide_designer/(Pipelex 0.95.1): "Successfully validated the method".run_6dcc9217-961d-41be-9f1e-bc081221bd86, drew three mockups all titled "Q4 2025 Financial Results", each in its own theme's palette; the sampleinputs.json,run_3de8f0a8-5744-476f-8696-76beab5400fc, kept the topic as typed and drew it as the title;generate_design_proposals_report,run_6a6de223-e8a7-4ef0-b41e-ed7c346be41d, rendered three cards, each mockup beside its own theme.compose_proposals_reportrendered alone from the first run's result: the "Enhanced Brief" block is hidden for an unchanged brief and shown for a topic-only one.Documentation
CHANGELOG.md[Unreleased]: the breaking change under Changed, the title fix under Fixed.README.md: theslide_designerrow says what the result pairs and names the report entry.Not done
pipelex-demos,pipelex-demo-frostbeamand mthds-form's input-form fixture are not fed from the library and keep the old image step./releaseskill.Closes L-261107-75b313
🤖 Generated with Claude Code
https://claude.ai/code/session_019TF9N3ixWPEiwAUJDxxxfd
Summary by cubic
Makes the slide designer draw the deck's own title and return each theme paired with its mockup.
The image step previously never saw the brief, so the image model invented a subject and period for the title slide; it now reads the polished brief and draws its
topicword for word, with the other slides' placeholder content fitting the topic, audience and goal. The polish step keeps the topic exactly as typed.Breaking change
generate_design_proposals_from_rough_briefnow returnsDesignProposals(the polished brief plus one theme-and-mockup pair per proposal) instead of the HTML page; callers use the newly exportedgenerate_design_proposals_reportfor the old page and stay on@v0.3.0to keep the old pipe.Written for commit 3f6b2e9. Summary will update on new commits.