AVO-2568: document how the import API decides "Include in Codegen" - #1738
AVO-2568: document how the import API decides "Include in Codegen"#1738ada-avo wants to merge 1 commit into
Conversation
The import now defaults a newly attached source to "Include in Codegen" whenever the plan allows codegen and the source is codegen-configured, matching what the UI does when you attach a source by hand. The docs said nothing about the setting, and the sources table left readers to guess what happened to the `implementWithCodegen` field the Avo export writes. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GkHYCAX9UD9sB9YFjBJmtp
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Limit details: You’ve used all 5 included reviews currently available under your plan. You completed 50 included PR reviews in the past 7 days; at that activity level, included reviews refill at 5 reviews per hour. 📝 WalkthroughWalkthroughThe import tracking plan documentation now explains that ChangesCodegen inclusion documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This localized documentation update clarifies existing import behavior and adds an anchor link without changing product behavior; no actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
pages/reference/public-api/import-tracking-plan.mdxParsing error: Unexpected token (1:0) Usage-based review receipt
Note This review was completed with usage-based billing: files reviewed beyond your plan's included limits are billed at $0.25/file. Track spend and usage in your billing settings. Comment |
📝 Changed routes:
|
What changed?
Pairs with avohq/monorepo#9953, which makes the tracking-plan import default a newly attached source to "Include in Codegen" whenever the workspace's plan allows codegen and the source is codegen-configured — the same rule the Avo UI applies when you attach a source by hand. Before that change the import always left codegen off, silently.
The import API reference said nothing about the setting. The
sourcesrow said only "Only thenamefield is extracted", which left readers to guess what happened to theimplementWithCodegenfield the Avo export writes into that same array — and the answer ("ignored, but the setting is still decided for you") is the thing a customer automating their plan needs to know.Two edits to
pages/reference/public-api/import-tracking-plan.mdx:save_itemstool or the UI — to change it on an already-attached source.sourcesrow in the field table now saysimplementWithCodegenis ignored and links to that section.How to test this PR:
Preview the Import Tracking Plan reference page: the new section renders between the parse-error list and Import Methods, and the anchor link from the
sourcesrow's Notes cell resolves to it.Every claim is checked against the implementation on the monorepo branch: the default comes from
IncludeSourceInCodegenDefaultUseCase.includeInCodegenByDefault(plan availability AND a source with a language, platform, and destination), the parser still reads onlynamefrom each source entry, andImportParserFilteringcarries only sources that are new to an event — which is what makes the "never overwritten" claim true.Generated by Claude Code
Summary by CodeRabbit
sourcesfield includes source names but ignores code generation settings.