Distillation

Date: Sep 8, 2026

• 🟡 (17:54) Inspected /home/byk/Code/getsentry/publish-workspace-acceptance/src/publish/__tests__/discover-location.js: imports afterEach, expect, test, and vi from vitest; requires discoverLocation and getWorkspaceNames from ../discover-location.js; afterEach() calls vi.restoreAllMocks().
• 🟡 (17:54) Existing discover-location.js regression test "retains checkout-path behavior when the root config is absent" calls discoverLocation({ input: { path: "./packages/cli" }, repositoryDirectory: "__repo__", exists: () => false }) and expects { path: "./packages/cli" }.
• 🟡 (17:54) Existing test "discovers exact workspace paths using the released Craft image" mocks execFile to return '["packages/CLI"]', calls discoverLocation() with input.path: "./packages/CLI", repositoryDirectory: "__repo__", and exists: () => true, expects { path: ".", workspace: "packages/CLI" }, and verifies docker is called with arguments containing "getsentry/craft:latest", "workspace", "list" and { encoding: "utf8" }.
• 🟡 (17:54) Existing test "fails closed when Craft returns an invalid workspace list" mocks Craft output as "{}" and expects getWorkspaceNames() to throw "Craft workspace discovery returned an invalid workspace list.".
• 🟡 (17:54) Existing test "fails closed when Craft returns no workspace output" mocks Craft output as "\n" and expects getWorkspaceNames() to throw "Craft workspace discovery returned an invalid workspace list.".
• 🟡 (17:54) Assistant narrowed the proposed fallback: path "." is never a workspace identity and therefore does not need workspace discovery; any non-root path will retain fail-closed workspace discovery.
• 🟡 (17:55) Created Publish hotfix branch fix/root-location-bootstrap, configured to track origin/main; its HEAD is 7c60ddb (feat: accept workspace publish requests).
• 🟡 (17:55) Assistant stated it will add a fail-first regression proving root publishing skips unavailable workspace discovery while non-root locations still fail closed.