[EPD-198] Document restoreFromStateId in kickoff API reference + migration guide operational notes#6541
Conversation
…ion guide operational notes - Add restoreFromStateId (optional, uuid) to the POST /kickoff request schema with fork semantics, silent fresh-session fallback for unknown ids, and a pointer to the migration guide - Add a flow_state_restore request example and a 422 example for non-UUID inputs.id alongside the existing missing_inputs case - Add an Operational notes section to the inputs-id-deprecation guide covering how to chain turns today (client-generated UUID pinned via inputs.id), the silent fresh-session behavior for unknown restoreFromStateId, and the AMP UUID format requirement for inputs.id Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
📝 WalkthroughWalkthroughThe AMP ChangesAMP kickoff state restoration
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/edge/en/guides/flows/inputs-id-deprecation.mdx`:
- Around line 156-159: Replace the non-UUID placeholder values in the request
example with valid UUID-formatted placeholders for both inputs.id and
restoreFromStateId, while preserving the example’s meaning and surrounding flow.
In `@docs/edge/enterprise-api.base.yaml`:
- Line 152: Replace the invalid UUID examples with a valid hexadecimal UUID
placeholder at docs/edge/enterprise-api.base.yaml lines 152-152 and 197-197, and
docs/edge/enterprise-api.en.yaml lines 152-152 and 197-197; update both the
example and restoreFromStateId values while leaving unrelated kickoff_id
occurrences unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 3976ae72-27a5-4a7d-87c7-1c203330ce29
📒 Files selected for processing (3)
docs/edge/en/guides/flows/inputs-id-deprecation.mdxdocs/edge/enterprise-api.base.yamldocs/edge/enterprise-api.en.yaml
- Replace the non-hex example UUID with a valid one in the restoreFromStateId property and flow_state_restore example added by this PR (pre-existing invalid example UUIDs elsewhere in the file are left as a broader cleanup) - Use valid, copy-pasteable UUID placeholders in the migration guide curl example with a comment noting they are examples, since AMP 422s non-UUID inputs.id values Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Addressed CodeRabbit feedback: the examples this PR added now use valid UUIDs (schema example, flow_state_restore request example, and the migration-guide curl placeholders). Note: the pre-existing example UUID pattern |
restoreFromStateId(optional string, uuid format) to the POST /kickoff request schema indocs/edge/enterprise-api.en.yaml(and the in-syncenterprise-api.base.yaml): documents fork semantics (hydrates from the referenced state, the new run gets a fresh state id), the silent fresh-session fallback when the id is unknown, and links to the migration guideflow_state_restorerequest example and document the 422 variant where a non-UUIDinputs.idis rejected ("'inputs.id' must be a valid UUID when provided; got 'test 1'.") alongside the existing missing_inputs casedocs/edge/en/guides/flows/inputs-id-deprecation.mdxcovering: the API does not yet return the minted state id (chain turns today by pinning a client-generated UUID viainputs.idalongsiderestoreFromStateId; gap is tracked), unknownrestoreFromStateIdsilently starts a fresh session with no failure signal, and AMP's UUID format requirement forinputs.iddocs/v*/snapshots and non-English locales untouched🤖 Generated with Claude Code
Note
Low Risk
Documentation-only changes to Edge guides and OpenAPI YAML; no runtime or API implementation changes.
Overview
Documents
restoreFromStateIdon POST/kickoffin the Edge OpenAPI specs (enterprise-api.en.yaml/enterprise-api.base.yaml): optional UUID to hydrate from a prior@persistflow with fork semantics, silent fresh session when the id is unknown, plus aflow_state_restoreexample. 422 is expanded to cover invalidinputs.id(non-UUID) with concrete response examples alongside missing inputs.The
inputs.iddeprecation guide gains an Operational notes section for AMP REST chaining: the API still does not return the new run’s minted state id (interim pattern of client UUID viainputs.id+restoreFromStateId), unknown restore ids fail open with no signal, and AMP requires UUID-shapedinputs.id.Reviewed by Cursor Bugbot for commit f90f735. Bugbot is set up for automated code reviews on this repo. Configure here.