From 16a5fb763eca0cb92f0d0413156acfb0d832b923 Mon Sep 17 00:00:00 2001 From: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Date: Thu, 25 Jun 2026 22:03:04 +0000 Subject: [PATCH] fix: update spike-orchestrator docs and exclude type-only exports from structural regex Co-Authored-By: mantra --- apps/backend/scripts/spike-orchestrator-e2e.mts | 4 ++-- packages/shared-backend/src/config-updater.ts | 4 +++- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/apps/backend/scripts/spike-orchestrator-e2e.mts b/apps/backend/scripts/spike-orchestrator-e2e.mts index 92c789834..1a8258d81 100644 --- a/apps/backend/scripts/spike-orchestrator-e2e.mts +++ b/apps/backend/scripts/spike-orchestrator-e2e.mts @@ -6,8 +6,8 @@ * snapshot if STACK_CONFIG_AGENT_BASE_SNAPSHOT_ID is set, else cold-install the * agent SDK), clone the repo, agent edits the config, COMMIT + PUSH to the branch. * - * WARNING: this pushes a commit to `main` of the target repo. Point it at a - * throwaway repo (default: hexclave/stackframe-website-2026). + * WARNING: this pushes a commit to the target repo. Point SPIKE_OWNER, + * SPIKE_REPO, and SPIKE_BRANCH at a throwaway repo/branch. * * Run from apps/backend: * pnpm dlx dotenv-cli -e .env.development.local -e .env.development -- \ diff --git a/packages/shared-backend/src/config-updater.ts b/packages/shared-backend/src/config-updater.ts index 7f471e56d..dbfa2cc85 100644 --- a/packages/shared-backend/src/config-updater.ts +++ b/packages/shared-backend/src/config-updater.ts @@ -190,7 +190,9 @@ function configFileExportsConfig(content: string, configFilePath: string): boole // packages). For the structural sanity check we only need to know a runtime // `config` binding still exists after the agent edited the file. Covers // `export const config`, `export let config`, `export { config }`, etc. - return /\bexport\s+(?:const|let|var)\s+config\b/.test(content) || /\bexport\s*\{[^}]*\bconfig\b/.test(content); + // Excludes type-only exports (`export { type config }`) which have no + // runtime binding. + return /\bexport\s+(?:const|let|var)\s+config\b/.test(content) || /\bexport\s*\{[^}]*(?