Skip to content

generate: stage local dependency closure before codegen - #2

Draft
tiborvass wants to merge 1 commit into
mainfrom
agent/generate-stage-local-dependencies
Draft

generate: stage local dependency closure before codegen#2
tiborvass wants to merge 1 commit into
mainfrom
agent/generate-stage-local-dependencies

Conversation

@tiborvass

@tiborvass tiborvass commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Uses the new ModuleSource.generateLocalDependencies engine API. Before generating each managed PHP module, generateAll (the @generate function) and Mod.generate stage the transitive local dependency closure onto the workspace via withChanges, then generate against the staged workspace. This lets PHP module codegen see up-to-date dependency bindings, including dependencies owned by other SDKs.

The existing github.com/dagger/polyfill@main dependency is advanced to the module-generation scoping fix, so each generated changeset contains files for only that module.

Depends on

This is the PHP SDK equivalent of dagger/typescript-sdk#12 and dagger/go-sdk#12.

Validation

  • jq empty dagger.json
  • (cd helpers/render-template && go test ./...)
  • git diff --check

A full generation test was not run locally because the installed stable CLI (v0.21.7) predates the CLI-1.0 engine API used here.

@tiborvass
tiborvass force-pushed the agent/generate-stage-local-dependencies branch from b198b29 to 10376b9 Compare July 29, 2026 16:15
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