@bridge_gpt/mcp-server 0.2.2 → 0.2.4

package/smoke-test/SMOKE-TEST.md

@@ -342,11 +342,16 @@ Ask for one explicit Tier 3 consent decision; if granted, the user may pick whic
 flows to run. Use **bounded polling with increasing intervals and a maximum wait
 of about 15 minutes** for any async flow.
 
-### Synchronous Tier 3 tool
+### Single-call Tier 3 tool
 
-- **`second_opinion` is synchronous and token-costing.** It blocks and returns
-  its result directly — it is **not** an async request/get flow. Do not try to
-  capture an ID for it.
+- **`second_opinion` returns its result in one call and is token-costing.** The
+  backend is now async (submit → poll → result, so a slow opinion no longer trips
+  the 30s Heroku request timeout / H12), but the tool **absorbs that polling
+  internally** — it still returns the reply directly. It is **not** an async
+  request/get flow: do not try to capture an ID for it. If an opinion is still
+  processing after ~5 minutes, the tool returns a **recoverable-result message**
+  (with the `request_id` and a `GET /llm/second-opinion/{id}/result` URL) rather
+  than failing — treat that as `PASS`, not an error.
 
 ### Async request/get Tier 3 tools
 
@@ -484,12 +489,12 @@ and raw protocol noise.** Sensitive values are redacted.
 
 ---
 
-## Full 56-Tool Smoke Tier Matrix
+## Full 62-Tool Smoke Tier Matrix
 
 Every registered MCP tool appears **exactly once** below with its smoke tier.
 Tier 4 (mutating/hazardous) tools are **deferred in v1**.
 
-Coverage reconciliation: **8 + 21 + 1 + 11 + 18 = 59**.
+Coverage reconciliation: **8 + 22 + 1 + 12 + 19 = 62**.
 
 | Tier | Tool | Default action / expected-empty / special handling |
 |---|---|---|
@@ -507,7 +512,8 @@ Coverage reconciliation: **8 + 21 + 1 + 11 + 18 = 59**.
 | Tier 1 | `get_comments` | Read-only; needs ticket key (preferred default-pass tool). |
 | Tier 1 | `get_plan` | Artifact retriever; `PASS` with content, `PASS_EXPECTED_404` when cleanly empty; boolean probe target. |
 | Tier 1 | `get_architecture` | Artifact retriever; `PASS_EXPECTED_404` when cleanly empty. |
-| Tier 1 | `get_design_doc` | Artifact retriever (tdd/fsd via `doc_type`); `PASS_EXPECTED_404` when cleanly empty. |
+| Tier 1 | `get_doc` | Artifact retriever (tdd/fsd/prd via `doc_type`); `PASS_EXPECTED_404` when cleanly empty. |
+| Tier 1 | `get_prd` | Artifact retriever (Product Requirements Document); `PASS_EXPECTED_404` when cleanly empty. |
 | Tier 1 | `get_install_manifest` | Read-only; returns the easy-install bootstrap manifest + signed snapshot token; a clean 404 (no config row) is `PASS_EXPECTED_404`. |
 | Tier 1 | `get_clarifying_questions` | Artifact retriever; `PASS_EXPECTED_404` when cleanly empty. |
 | Tier 1 | `get_ticket_critique` | Artifact retriever; `PASS_EXPECTED_404` when cleanly empty. |
@@ -523,11 +529,12 @@ Coverage reconciliation: **8 + 21 + 1 + 11 + 18 = 59**.
 | Tier 1 | `poll_ci_checks` | `available:true` with empty or populated checks is `PASS`; `SKIP_NOT_APPLICABLE` when disabled with `TOOL_DISABLED`/503, or when `resolve_ci_checks` has not populated CI configuration. |
 | Tier 1 | `get_pipeline_recipe` | Read-only; needs a pipeline name plus required variables for variable-declaring pipelines; use `learn-repository` as the no-variable smoke target. |
 | Tier 2 | `generate_decision_page` | Local-only file-write canary; always run with the verbatim payload above. |
-| Tier 3 | `second_opinion` | Synchronous, token-costing; opt-in. |
+| Tier 3 | `second_opinion` | Single-call (absorbs async polling under the hood — H12-safe); token-costing; opt-in. A ~5-min recoverable-result message is `PASS`, not a failure. |
 | Tier 3 | `generate_image` | Synchronous, token/credit-costing; spends provider credits; opt-in. |
 | Tier 3 | `request_plan_generation` | Async (ticket-key based); retrieve with `get_plan`; opt-in. |
 | Tier 3 | `request_architecture` | Async (ticket-key based); retrieve with `get_architecture`; opt-in. |
-| Tier 3 | `request_design_doc` | Async (ticket-key based; `doc_type` tdd/fsd); retrieve with `get_design_doc`; opt-in. |
+| Tier 3 | `create_doc` | Async (ticket-key based; `doc_type` tdd/fsd/prd); retrieve with `get_doc`; opt-in. |
+| Tier 3 | `request_prd` | Async (ticket-key based; Product Requirements Document); retrieve with `get_prd`; opt-in. |
 | Tier 3 | `request_clarifying_questions` | Async (ticket-key based); retrieve with `get_clarifying_questions`; opt-in. |
 | Tier 3 | `request_ticket_critique` | Async (ticket-key based); retrieve with `get_ticket_critique`; opt-in. |
 | Tier 3 | `request_ticket_review` | Async submit; retrieve with `get_clarifying_questions` + `get_ticket_critique`; opt-in. |
@@ -552,3 +559,4 @@ Coverage reconciliation: **8 + 21 + 1 + 11 + 18 = 59**.
 | Tier 4 | `run_full_automation` | Orchestration (dispatches mutating child pipelines / spawns worktrees); deferred in v1. |
 | Tier 4 | `resume_full_automation` | Orchestration; deferred in v1. |
 | Tier 4 | `apply_install_manifest` | Mutating (atomically writes easy-install config across the four config tables); deferred in v1. |
+| Tier 4 | `persist_routing_credential` | Mutating (writes the routing credential to the user-scoped store `~/.config/bridge/credentials.json`); deferred in v1. |