feature/Starter-reference-drift · L-260913-84dd45 — stale picture of the Python starter - #24
Merged
Merged
Conversation
…grate references The references said both `make codegen` and `make codegen-check` need a `pipelex` CLI, so on a starter-derived project the integrate skill took the workshop fallback when the starter's own generator would have worked. `make codegen` now goes through the hosted API with `pipelex-sdk` and needs only a key, a method directory may hold a `method.json` instead of a bundle, and the starter carries `AGENTS.md` and `docs/add-method.md`. The fallback now branches on the key reaching the project, the manifest path is spelled out, and tests pin both. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AcWGJs6kVUxGzHEKShjbFj
…ule name The harness section reused the method directory's dashed name for the `__init__.py` path, the workshop fallback's `output_dir` and the model import, so a kebab-case method would get a stray directory or a second tree. Those paths now use `<module>`, the name with dashes turned into underscores, and the bundle bullet says the entry file is `main.mthds`, which the starter's tests open. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AcWGJs6kVUxGzHEKShjbFj
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 scaffold and integrate references described pipelex-starter-python before its hosted-codegen and agent-surface work. They said `make codegen` needs a `pipelex` CLI, which sent the integrate skill down the workshop fallback when the starter's own generator would have worked, and they knew nothing of `method.json`, `make add-method`, `AGENTS.md` or `docs/add-method.md`. They now match the starter's dev. The workshop fallback branches on a key reaching the project, the generated paths use the underscored module name, and tests pin both. The references describe the starter's dev while scaffolding clones its main, so this is accurate for new projects once the starter's next release is cut.
Closes L-260913-84dd45
🤖 Generated with Claude Code
https://claude.ai/code/session_01AcWGJs6kVUxGzHEKShjbFj
Summary by cubic
Updates the scaffold and integrate references to match the current
pipelex-starter-python, so the integrate skill stops taking the workshop fallback when the starter's own generator would work.Bug Fixes
make codegennow regenerates through the hosted API withpipelex-sdkand needs onlyPIPELEX_API_KEY; the workshop fallback branches on a missing key instead of a missingpipelexCLI.method.json(exactly one ofmethod_id/method_ref) instead of a bundle, andmake add-methodscaffolds it.summarize-pdf→summarize_pdf), never the dashed name.AGENTS.md,docs/add-method.md, and theCLI_INPUTSentry thatmake agent-testrequires.pipelexCLI branching.The references describe the starter's dev branch, so they are accurate for new projects once the starter's next release is cut.
Closes L-260913-84dd45
Written for commit 62709d8. Summary will update on new commits.