Skip to content

parity(workspaces): fix all parity.md findings + teststack backend wiring#2348

Merged
agbishop merged 2 commits into
mainfrom
polecat/amber/go-8189a
Jun 24, 2026
Merged

parity(workspaces): fix all parity.md findings + teststack backend wiring#2348
agbishop merged 2 commits into
mainfrom
polecat/amber/go-8189a

Conversation

@agbishop

Copy link
Copy Markdown
Collaborator

WorkSpaces parity fixes. Polecat (gemini-pro/agy) validated: unit + e2e + integration green. Lands via PR (GH006-safe).

🤖 Generated with Claude Code

@agbishop agbishop enabled auto-merge (squash) June 24, 2026 09:21
amber and others added 2 commits June 24, 2026 08:29
- Add region field to storedWorkspace; store via regionFor(ctx) on CreateWorkspace
- Filter workspaces by region in DescribeWorkspaces
- Validate DirectoryID is registered in CreateWorkspace
- Add ctx param to CreateWorkspace, DescribeWorkspaces, DescribeWorkspaceBundles,
  DescribeWorkspaceDirectories; pass through from handlers
- Add cursor-based pagination to DescribeWorkspaceBundles (page size 25)
- Add cursor-based pagination to DescribeWorkspaceDirectories (page size 50)
- Convert hardcodedBundles() to pre-sorted amazonBundleList() function
- Refactor handleCreateWorkspaces into validateCreateWorkspacesInput,
  specToCreationSpec, wsToPending helpers to stay under funlen threshold
- Update existing tests to register directories before CreateWorkspaces
- Add backend_parity_test.go with 5 table-driven parity tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@agbishop agbishop force-pushed the polecat/amber/go-8189a branch from 10b232e to 6493767 Compare June 24, 2026 13:31
@agbishop

Copy link
Copy Markdown
Collaborator Author

📊 Code Coverage Report

Metric Value Status
Total Coverage 0.0%
82.3%
New Code Coverage N/A (0/0 stmts)

Tip

This project maintains a minimum coverage threshold of 85%. Maintain or improve coverage on new code to ensure long-term stability.


Last updated: Wed, 24 Jun 2026 13:52:08 GMT

@agbishop agbishop merged commit a23f85b into main Jun 24, 2026
25 checks passed
@agbishop agbishop deleted the polecat/amber/go-8189a branch June 24, 2026 13:54
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