feat: unify issue search semantics#67
Conversation
Codename-11
left a comment
There was a problem hiding this comment.
Self-review complete: no actionable findings. Reviewed shared identifier parsing/predicate composition, API-key narrowing, workspace/lifecycle/archive/facet composition, deterministic pagination, MCP/command-palette parity, and the focused regression coverage. GitHub CI is green across lint/typecheck/unit and both Playwright shards.
a3105b5 to
536fb69
Compare
Codename-11
left a comment
There was a problem hiding this comment.
Updated-head review complete at 536fb69 after rebasing onto v0.27.1 / 684df53: no actionable findings. Re-reviewed shared identifier parsing, tenant and API-key scope composition, metadata predicates, deterministic ordering/cursor boundaries, MCP and command-palette parity, and regression coverage. Local lint, typecheck, 1,425-test Vitest gate, fresh production build, and focused Playwright journeys pass.
Summary
Verification
pnpm lint(passes; pre-existing warnings only)pnpm typecheckpnpm test --no-file-parallelismwith CI-equivalent local Postgres, Redis, and MinIO settings (183 files passed, 1 skipped; 1,425 tests passed, 1 skipped)pnpm exec playwright test tests/e2e/issues-scope.spec.ts --workers=1(2 passed against dedicated localforge_e2eDB / Redis DB 15)git diff --check origin/main...HEADSafety
Implements AXI-119.