fix: remove stale main-only team-state test and franken-merged beforeAll artifact

- test/e2e-runbook-hygiene.test.ts: deliberately deleted from dev in PR #2015
  (Isolate Squad product assets from team state) because it read live .squad
  state. It survived only on main (added at v0.13.0, no promotion since), so
  the dev-vs-main merge resurrected it. Removing it to match dev's documented,
  traceable intent — not a blanket ours/theirs choice.
- test/template-sync.test.ts: the 3-way merge auto-combined non-overlapping
  hunks from dev (simplified describe block, no beforeAll) and main (legacy
  describe-scoped beforeAll retained for historical defence-in-depth) into
  semantically broken code referencing an unimported beforeAll. Replaced with
  dev's complete, authoritative file content (byte-identical to origin/dev).

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: tamirdresher

test/e2e-runbook-hygiene.test.ts

@@ -110,20 +110,6 @@ const CASTING_REGISTRY_LOCATIONS = [
 // ---------------------------------------------------------------------------
 
 describe('dynamic template enumeration (all synced files)', () => {
-  // Re-sync immediately before byte comparisons as a second layer of defence.
-  // The original race — test/init-scaffolding.test.ts running runInit() against
-  // a sandbox under the repo root, so monorepo resolution stamped the real
-  // .github/agents/squad.agent.md — was fixed in #1796 by moving that sandbox
-  // to the OS temp dir. This describe-scoped beforeAll is retained to keep the
-  // window closed if any future suite reintroduces an in-repo sandbox.
-  beforeAll(() => {
-    execSync('node scripts/sync-templates.mjs', {
-      cwd: ROOT,
-      encoding: 'utf-8',
-      timeout: 60_000,
-    });
-  });
-
   const sourceFiles = collectFiles(SOURCE_DIR);
 
   it('.squad-templates/ contains files to sync', () => {