Skip to content

feat(agent-gui): add canonical session rail placement#1565

Merged
SingleMai merged 1 commit into
mainfrom
codex/shared-agent-canonical-rail-placement
Jul 23, 2026
Merged

feat(agent-gui): add canonical session rail placement#1565
SingleMai merged 1 commit into
mainfrom
codex/shared-agent-canonical-rail-placement

Conversation

@SingleMai

@SingleMai SingleMai commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

What changed

  • add a versioned canonical rail-placement value to AgentGUI session creation and shared-agent Host contracts
  • persist the caller-selected kind, projectPath, and opaque sectionKey as immutable session data
  • reject idempotent retries whose supplied placement differs from canonical state
  • preserve placement across owner hosting and subsequent session reads
  • keep placement optional for mixed-version clients; no pre-launch history backfill

Why

Shared-agent placement belongs to the caller's current user_project, not the owner's local project mapping. The persisted rail identity also lets project deletion retain current Tutti deleted-project semantics without relocating old sessions to Conversations.

Coordinated rollout

This PR remains draft. After merge it needs a Tutti release; TSH must then upgrade all aligned Tutti dependencies and generated files atomically.

Verification

  • pnpm generate:api (generated output clean)
  • pnpm check:changed -- --push-ready (32 lanes passed)
  • Host/store/service conformance covers same-key/different-project retry conflicts
  • pre-commit formatting and repository boundary checks

@SingleMai
SingleMai force-pushed the codex/shared-agent-canonical-rail-placement branch from 670d34e to 4c24290 Compare July 23, 2026 09:53
Signed-off-by: liying <422107224@qq.com>
@SingleMai
SingleMai force-pushed the codex/shared-agent-canonical-rail-placement branch from 4c24290 to 48b1345 Compare July 23, 2026 10:36
@SingleMai
SingleMai marked this pull request as ready for review July 23, 2026 11:19

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@SingleMai
SingleMai merged commit 663ba3b into main Jul 23, 2026
14 checks passed
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