Skip to content

feature/Starter-reference-drift · L-260913-84dd45 — stale picture of the Python starter - #24

Merged
lchoquel merged 2 commits into
devfrom
feature/Starter-reference-drift
Sep 13, 2026
Merged

lchoquel merged 2 commits into
devfrom
feature/Starter-reference-drift

Conversation

@lchoquel

@lchoquel lchoquel commented Sep 13, 2026

Copy link
Copy Markdown
Member

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 codegen now regenerates through the hosted API with pipelex-sdk and needs only PIPELEX_API_KEY; the workshop fallback branches on a missing key instead of a missing pipelex CLI.
  • A method directory can hold a method.json (exactly one of method_id / method_ref) instead of a bundle, and make add-method scaffolds it.
  • Generated directories and model imports use the underscored module name (summarize-pdfsummarize_pdf), never the dashed name.
  • References now list AGENTS.md, docs/add-method.md, and the CLI_INPUTS entry that make agent-test requires.
  • Tests pin the new wording and forbid the old pipelex CLI 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.

Review in cubic

lchoquel and others added 2 commits September 14, 2026 01:44
…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
@lchoquel
lchoquel merged commit 426c46c into dev Sep 13, 2026
2 checks passed
@lchoquel
lchoquel deleted the feature/Starter-reference-drift branch September 13, 2026 23:57
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