[AI-7174] Remove manifest-less args from ddev create tool - #25221
Conversation
|
✅ All CI checks and tests passed. 🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: bee74d7 | Docs | View more details | Give us feedback! |
evalya-impact-summaryevalya impact analysis |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 385af22998
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
385af22 to
bff624f
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bff624f0df
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
ddev create now normalizes the raw display name it is given to snake_case for the directory/package/metrics prefix while keeping the display name verbatim in the manifest. Pass the display name straight through the ddev_create tool instead of pre-normalizing it in flow prompts, and describe that behavior on the tool's input field so every flow gets it for free.
Validation ReportAll 21 validations passed. Show details
|
What does this PR do?
Remove the manifest-less args from the
ddev_createtool in Togo.The tool's
integrationinput now carries the display name straight through toddev create: the command itself normalizes it to snake_case for the directory, Python package, and metrics prefix, while preserving the display name verbatim in the manifest's display-name fields. That behavior is documented on the tool's input field so any flow using the tool gets it for free, instead of every flow having to know to skip pre-normalizing the name before scaffolding.Motivation
New integrations restore the
manifest.jsonfile, andddev createcommand has changed in #25218, so we need to update the tool here as well.Review checklist (to be filled by reviewers)
qa/requiredif this PR needs QA validation, orqa/skip-qaif it does not. Exactly one of the two is required.backport/<branch-name>label to the PR and it will automatically open a backport PR once this one is merged