fix: persist active workspace and redirect on session/workspace mismatch - #238

…249)

## Summary

`main` (commit `8cc8ff39`) is red, blocking the **v0.0.6** release PR
#248. All three failing release-PR checks trace back to `main`, not the
release branch:

- **Verify** — `internal/daemon/coordinator_runtime.go` `bootstrapRun`
exceeded the gocyclo threshold (CC **21 > 20**). Extracted
`reconcileCreatedCoordinator` and `wakeCoordinatorIfNeeded` — a
behavior-preserving split that drops complexity to ~10.
- **Integration** —
`TestHarnessContextIntegrationStartupAndPromptShareResolverPolicy`
counted `<agh-situation-context>` (the **open** tag), which the bundled
tools guide (`tools-and-skills.md`) documents in prose, so the count was
2. It now counts the closing `</agh-situation-context>` tag → measures
**rendered sections** (exactly 1). The product already renders the
section once; the assertion was fragile (red since v0.0.5).
- **E2E Nightly** — the sandbox *"real session execution"* test
direct-navigates to a session created in a temp workspace while the
**global** workspace is active. #238's `useSessionWorkspaceGuard`
redirects away on a workspace/session mismatch, so the composer never
rendered and `locator.fill` hit the 180s timeout. The test now activates
the session's workspace via the switcher first — the
persisted-active-workspace precondition #238 assumes for direct
navigation.

## Verification

- `make verify` — **pass** (full monorepo gate).
- golangci-lint **v2.12.2** `./...` (CI's pinned version) — **0
issues**.
- `go test -race ./internal/daemon/...` — **875 unit pass**; **987
integration pass** (`-tags integration`).
- `bun-lint` — 0 warnings / 0 errors; web typecheck — pass.
- The E2E fix is verified by deterministic root-cause analysis +
typecheck/lint; the release PR's **E2E Nightly** lane (the lane that
failed) is its end-to-end gate. A faithful single-test local run is
blocked by the nightly coverage-gate plus the daemon-served harness env.

## Release impact

Merging this refreshes `release/v0.0.6` (#248) to include the fix so its
**Verify** and **Integration** lanes go green. Version stays **v0.0.6**
— CI's pinned git-cliff `2.7.0` treats `feat:` as a patch bump on `0.x`
(same as how v0.0.5 was cut).

> Note: v0.0.5's own release PR (#237) merged with **Integration +
Verify failing**. The heavy lanes (Integration, E2E Nightly) only run on
release PRs, so these regressions slipped past per-feature CI. Worth
tightening the merge gate separately.

AGH Impact Audit:

- Native tools: no impact — no `agh__*` IDs, toolsets, descriptors,
input/output schemas, schema digests, risk flags, or capability gates
changed; this is an internal daemon refactor plus two test files.
- Extensibility and hooks: no impact — no extension manifests/resources,
hook taxonomy or dispatch call sites, skill/capability declarations,
bundles, registries, bridge SDKs, MCP sidecars, or `config.toml`
lifecycle changed. `bootstrapRun`'s dispatch call sites, locking, and
ordering are preserved by the extraction.
- Workspace data isolation: no impact on runtime data ownership — the
coordinator refactor preserves `workspace_id`-scoped
bootstrap/reconcile/dispatch exactly; the e2e change only makes the test
honor the existing per-workspace session-route guard from #238.
- Official AGH skill: no impact — `skills/agh/SKILL.md` and
`skills/agh/references/*.md` are unchanged. The `tools-and-skills.md`
`<agh-situation-context>` mention is correct documentation and
intentionally left as-is; the test was corrected instead.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Refactor**
* Improved internal coordinator session initialization and management
code organization for better maintainability and clarity.

* **Tests**
* Updated integration tests with refined assertions to better validate
rendered output correctness.
* Enhanced end-to-end tests with explicit verification steps for
workspace-to-session binding workflows.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Co-authored-by: Claude Opus 4.8 (1M context) <[email protected]>