@bridge_gpt/mcp-server 0.2.0 → 0.2.2

package/pipelines/implement-ticket.json

@@ -22,7 +22,7 @@
     },
     {
       "type": "agent_task",
-      "instruction_file": "execute-plan-sectioned.md",
+      "instruction_file": "execute-plan.md",
       "description": "Execute the implementation plan"
     },
     {
@@ -58,31 +58,5 @@
       "instruction_file": "monitor-ci-checks.md",
       "description": "Monitor CI checks and report results"
     }
-  ],
-  "tiered_executor_policy": {
-    "supported_hosts": ["claude_code"],
-    "sequential_only": true,
-    "tier_model_mapping": {
-      "cheap": "haiku",
-      "basic": "sonnet",
-      "premium": "opus"
-    },
-    "final_review_policy": {
-      "default": "premium_whole_diff_when_below_coordinator_tier_touched_code",
-      "skip_when": "entire_run_inline_default_at_coordinator_tier"
-    },
-    "escalation_policy": {
-      "max_section_escalations": 1,
-      "allowed_hops": {
-        "cheap": "basic",
-        "basic": "premium"
-      },
-      "final_review_fix_reverify_passes": 1
-    },
-    "budget_policy": {
-      "cache_hit_rate_source": "measurement_spike_go_marker",
-      "default_cache_hit_rate": 0.0,
-      "abort_mode": "inline_default"
-    }
-  }
+  ]
 }

package/smoke-test/SMOKE-TEST.md

@@ -85,7 +85,10 @@ Ask the user for the following before making any tool calls:
    to be readable — comments, attachments, tracked state, and generated
    artifacts are all optional.
 2. **Optional:** a **config field name** (for `get_config_field`).
-3. **Optional:** a **pipeline name** (for `get_pipeline_recipe`).
+3. **Optional:** a **pipeline name** (for `get_pipeline_recipe`) — either a
+   no-variable pipeline name (e.g. `learn-repository`) or, for a
+   variable-declaring pipeline, the variable values it needs (e.g. a
+   `ticket_key`).
 4. **Optional:** an **attachment identifier or filename** (for
    `download_attachment` / `list_attachments`).
 5. **Optional:** any **artifact or task IDs** (e.g. a `task_id`, `brainstorm_id`,
@@ -179,6 +182,24 @@ all local writes:
 - A **host boolean-serialization rejection during the dedicated boolean probe**
   (Phase 7) is a **`WARN`** for host behavior — **not** an MCP server `FAIL`.
 
+### Heavy-read truncation grading
+
+There are **five heavy-read tools** whose successful oversized output is saved to
+disk before a truncated inline preview is returned: `get_project_standards`,
+`get_tickets`, `get_ticket`, `get_comments`, and `list_attachments`.
+
+- When a heavy-read response begins with `[Response truncated. Full response
+  saved locally.]` and carries a `Saved to <path>` note, grade it **`PASS`** as
+  long as the saved `<path>` is contained under the `get_docs_dir` result. The
+  truncation banner is the **expected** large-payload behavior, never a `FAIL`.
+- `get_project_standards` is a zero-input read that may exceed the inline
+  threshold; when it does, the full payload is saved under a `project-standards/`
+  subdirectory of the docs directory and only a truncated preview is returned
+  inline.
+- To exercise the truncation banner live, point the run at a known-large fixture
+  — a busy ticket with long comments, a broad `get_tickets` query, or a repo with
+  large project standards.
+
 ---
 
 ## Phase 3 — Default Pass
@@ -226,14 +247,32 @@ supplied the relevant input; otherwise record `SKIP_USER_INPUT`.
 - `get_ticket_state`, `get_jira_transitions`, `resolve_target_status` — the
   ticket key.
 - `get_config_field` — the optional config field name.
-- `get_pipeline_recipe` — the optional pipeline name.
+- `get_pipeline_recipe` — a pipeline name. When no variables are supplied, use a
+  no-variable pipeline such as `learn-repository` as the smoke target. A
+  variable-declaring pipeline (e.g. a ticket-based pipeline) requires a
+  `variables` map such as `{ "ticket_key": "<user ticket>" }`. A `400
+  BAD_REQUEST` with `Missing required variable(s)` is **working as designed**, not
+  a server fault — rerun with the required variables or pick a no-variable
+  pipeline.
 - `list_attachments` / `download_attachment` — the optional attachment input.
-- The artifact retrievers `get_architecture`, `get_clarifying_questions`,
-  `get_ticket_critique`, `get_reimplement_context`, `get_deep_research`,
-  `get_brainstorm` — grade a clean empty result as `PASS_EXPECTED_404` and valid
-  content as `PASS`.
-- `poll_ci_checks` — see the matrix note; `SKIP_NOT_APPLICABLE` when CI is not
-  configured.
+- The clean-empty artifact retrievers `get_architecture`,
+  `get_clarifying_questions`, `get_ticket_critique` — grade a clean empty result
+  as `PASS_EXPECTED_404` and valid content as `PASS`.
+- `get_reimplement_context` — can return **live-assembled, non-empty content**
+  even for an untracked or previously unprocessed ticket. Grade valid returned
+  content as `PASS`; a clean 404 is `PASS_EXPECTED_404`. It is **not** a
+  clean-empty-only retriever.
+- `get_ticket_state` — in a v1 read-only run this can only confirm the clean
+  "not tracked" 404 path unless a prior state-populating submit has occurred.
+  Full lifecycle-state coverage requires `track_ticket`, which remains a deferred
+  Tier 4 mutating tool.
+- `get_deep_research` — requires a `task_id` produced by `request_deep_research`.
+  When Tier 3 is skipped and no ID exists, record `SKIP_USER_INPUT`.
+- `get_brainstorm` — requires a `brainstorm_id` produced by `request_brainstorm`.
+  When Tier 3 is skipped and no ID exists, record `SKIP_USER_INPUT`.
+- `poll_ci_checks` — an `available:true` envelope with `checks:[]`,
+  `all_complete:true`, and `all_passed:true` is a `PASS`. The `TOOL_DISABLED` /
+  `503` self-disable branch is `SKIP_NOT_APPLICABLE` when CI is not configured.
 
 ---
 
@@ -445,46 +484,50 @@ and raw protocol noise.** Sensitive values are redacted.
 
 ---
 
-## Full 55-Tool Smoke Tier Matrix
+## Full 56-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 + 18 + 1 + 10 + 18 = 55**.
+Coverage reconciliation: **8 + 21 + 1 + 11 + 18 = 59**.
 
 | Tier | Tool | Default action / expected-empty / special handling |
 |---|---|---|
 | Tier 0 | `ping` | Zero-input read; always run. |
 | Tier 0 | `get_my_role` | Zero-input read; always run. |
 | Tier 0 | `get_docs_dir` | Zero-input read; always run; call first to anchor writes. |
-| Tier 0 | `get_project_standards` | Zero-input read; always run. |
+| Tier 0 | `get_project_standards` | Zero-input heavy read; always run; may save/truncate oversized output under `project-standards/`. |
 | Tier 0 | `list_config_fields` | Zero-input read; always run. |
 | Tier 0 | `list_pipelines` | Zero-input read; always run. |
 | Tier 0 | `get_parse_status` | Zero-input read; always run. |
 | Tier 0 | `list_pipeline_runs` | Zero-input read; always run. |
 | Tier 1 | `get_tickets` | Read-only; needs a query/filter. |
 | Tier 1 | `get_ticket` | Read-only; needs ticket key (default-pass tool). |
+| Tier 1 | `get_ticket_model_tier` | Read-only; needs ticket key; returns a coarse model tier (`cheap`/`basic`/`premium`) for start-tickets routing — `source:"fallback"` with a null tier is a clean `PASS`. |
 | 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_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. |
-| Tier 1 | `get_reimplement_context` | Artifact retriever; `PASS_EXPECTED_404` when cleanly empty. |
-| Tier 1 | `get_deep_research` | Artifact retriever; `PASS_EXPECTED_404` when cleanly empty. |
-| Tier 1 | `get_brainstorm` | Artifact retriever; `PASS_EXPECTED_404` when cleanly empty. |
-| Tier 1 | `get_ticket_state` | Read-only; needs ticket key. |
+| Tier 1 | `get_reimplement_context` | Artifact retriever; valid live-assembled content is `PASS`; a clean 404 is `PASS_EXPECTED_404` — the clean 404 is not the only expected outcome. |
+| Tier 1 | `get_deep_research` | Artifact retriever; requires a Tier 3-produced `task_id`; `PASS_EXPECTED_404` when cleanly empty. |
+| Tier 1 | `get_brainstorm` | Artifact retriever; requires a Tier 3-produced `brainstorm_id`; `PASS_EXPECTED_404` when cleanly empty. |
+| Tier 1 | `get_ticket_state` | Read-only; needs ticket key; v1 read-only reaches the clean "not tracked" 404; full state requires deferred Tier 4 `track_ticket`. |
 | Tier 1 | `get_jira_transitions` | Read-only; needs ticket key. |
 | Tier 1 | `resolve_target_status` | Read-only; needs ticket key. |
 | Tier 1 | `get_config_field` | Read-only; needs a config field name. |
 | Tier 1 | `list_attachments` | Read-only; needs ticket key; comments-fallback tool. |
 | Tier 1 | `download_attachment` | Tier 1 network read; secondary **path-canary** — saves locally only when an attachment input is supplied; omit `file_path` or keep any override under `get_docs_dir`. |
-| Tier 1 | `poll_ci_checks` | `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. |
+| 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 | `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 | `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. |
@@ -508,4 +551,4 @@ Coverage reconciliation: **8 + 18 + 1 + 10 + 18 = 55**.
 | Tier 4 | `regenerate_directory_map` | Hazardous; deferred in v1. |
 | 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 | `record_tiered_section_metric` | Writes server-side telemetry state (tiered_section_metrics row); deferred in v1. |
+| Tier 4 | `apply_install_manifest` | Mutating (atomically writes easy-install config across the four config tables); deferred in v1. |