Skip to content

fix: unify workspace authorization contract - #24

Merged
spytensor merged 1 commit into
mainfrom
codex/workspace-authorization-contract
Aug 5, 2026
Merged

spytensor merged 1 commit into
mainfrom
codex/workspace-authorization-contract

Conversation

@spytensor

Copy link
Copy Markdown
Owner

What changed

  • Resolve filesystem read and write roots through one authorization contract.
  • Bind runtime and delegated system prompts to the authenticated user's canonical workspace.
  • Keep Full Access read/write behavior symmetric for the configured shared workspace while preserving delegated hard pins and sensitive-write protection.
  • Return filesystem roots for the authenticated user instead of a process-global default.

Root cause

The runtime advertised one workspace path, writes admitted that path at Full Access, but reads/listing/image analysis resolved a different per-user allowlist. A task could therefore write a deliverable successfully and then be denied when verifying or reopening the exact same file.

Validation

  • Targeted Vitest: 5 files, 61 tests passed, including UUID-user write-then-read coverage.
  • pnpm typecheck
  • pnpm build
  • pnpm verify:public-export

@spytensor
spytensor merged commit 73b0ff4 into main Aug 5, 2026
2 of 3 checks passed
@spytensor
spytensor deleted the codex/workspace-authorization-contract branch August 5, 2026 11:26
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