oh-my-opencode 5.0.0-beta.8 → 5.0.0-beta.80

package/.opencode/skills/work-with-pr-workspace/iteration-1/eval-4/with_skill/outputs/code-changes.md

@@ -1,143 +0,0 @@
-# Code Changes: Issue #100 - Built-in arXiv MCP
-
-## 1. NEW FILE: `src/mcp/arxiv.ts`
-
-```typescript
-export const arxiv = {
-  type: "remote" as const,
-  url: "https://mcp.arxiv.org",
-  enabled: true,
-  oauth: false as const,
-}
-```
-
-Pattern: identical to `grep-app.ts` (static export, no auth, no config factory needed).
-
-## 2. MODIFY: `src/mcp/types.ts`
-
-```typescript
-import { z } from "zod"
-
-export const McpNameSchema = z.enum(["websearch", "context7", "grep_app", "arxiv"])
-
-export type McpName = z.infer<typeof McpNameSchema>
-
-export const AnyMcpNameSchema = z.string().min(1)
-
-export type AnyMcpName = z.infer<typeof AnyMcpNameSchema>
-```
-
-Change: add `"arxiv"` to `McpNameSchema` enum.
-
-## 3. MODIFY: `src/mcp/index.ts`
-
-```typescript
-import { createWebsearchConfig } from "./websearch"
-import { context7 } from "./context7"
-import { grep_app } from "./grep-app"
-import { arxiv } from "./arxiv"
-import type { OhMyOpenCodeConfig } from "../config/schema"
-
-export { McpNameSchema, type McpName } from "./types"
-
-type RemoteMcpConfig = {
-  type: "remote"
-  url: string
-  enabled: boolean
-  headers?: Record<string, string>
-  oauth?: false
-}
-
-export function createBuiltinMcps(disabledMcps: string[] = [], config?: OhMyOpenCodeConfig) {
-  const mcps: Record<string, RemoteMcpConfig> = {}
-
-  if (!disabledMcps.includes("websearch")) {
-    mcps.websearch = createWebsearchConfig(config?.websearch)
-  }
-
-  if (!disabledMcps.includes("context7")) {
-    mcps.context7 = context7
-  }
-
-  if (!disabledMcps.includes("grep_app")) {
-    mcps.grep_app = grep_app
-  }
-
-  if (!disabledMcps.includes("arxiv")) {
-    mcps.arxiv = arxiv
-  }
-
-  return mcps
-}
-```
-
-Changes: import `arxiv`, add conditional block.
-
-## 4. NEW FILE: `src/mcp/arxiv.test.ts`
-
-```typescript
-import { describe, expect, test } from "bun:test"
-import { arxiv } from "./arxiv"
-
-describe("arxiv MCP configuration", () => {
-  test("should have correct remote config shape", () => {
-    // given
-    // arxiv is a static export
-
-    // when
-    const config = arxiv
-
-    // then
-    expect(config.type).toBe("remote")
-    expect(config.url).toBe("https://mcp.arxiv.org")
-    expect(config.enabled).toBe(true)
-    expect(config.oauth).toBe(false)
-  })
-})
-```
-
-## 5. MODIFY: `src/mcp/index.test.ts`
-
-Changes needed:
-- Test "should return all MCPs when disabled_mcps is empty": add `expect(result).toHaveProperty("arxiv")`, change length to 4
-- Test "should filter out all built-in MCPs when all disabled": add `"arxiv"` to disabledMcps array, add `expect(result).not.toHaveProperty("arxiv")`
-- Test "should handle empty disabled_mcps by default": add `expect(result).toHaveProperty("arxiv")`, change length to 4
-- Test "should only filter built-in MCPs, ignoring unknown names": add `expect(result).toHaveProperty("arxiv")`, change length to 4
-
-New test to add:
-
-```typescript
-test("should filter out arxiv when disabled", () => {
-  // given
-  const disabledMcps = ["arxiv"]
-
-  // when
-  const result = createBuiltinMcps(disabledMcps)
-
-  // then
-  expect(result).toHaveProperty("websearch")
-  expect(result).toHaveProperty("context7")
-  expect(result).toHaveProperty("grep_app")
-  expect(result).not.toHaveProperty("arxiv")
-  expect(Object.keys(result)).toHaveLength(3)
-})
-```
-
-## 6. MODIFY: `src/mcp/AGENTS.md`
-
-Add row to built-in MCPs table:
-
-```
-| **arxiv** | `mcp.arxiv.org` | None | arXiv paper search |
-```
-
-## Files touched summary
-
-| File | Action |
-|------|--------|
-| `src/mcp/arxiv.ts` | NEW |
-| `src/mcp/arxiv.test.ts` | NEW |
-| `src/mcp/types.ts` | MODIFY (add enum value) |
-| `src/mcp/index.ts` | MODIFY (import + conditional block) |
-| `src/mcp/index.test.ts` | MODIFY (update counts + new test) |
-| `src/mcp/AGENTS.md` | MODIFY (add table row) |

package/.opencode/skills/work-with-pr-workspace/iteration-1/eval-4/with_skill/outputs/execution-plan.md

@@ -1,82 +0,0 @@
-# Execution Plan: Issue #100 - Built-in arXiv MCP
-
-## Phase 0: Setup
-
-1. `git fetch origin dev`
-2. `git worktree add ../omo-wt/feat/arxiv-mcp origin/dev`
-3. `cd ../omo-wt/feat/arxiv-mcp`
-4. `git checkout -b feat/arxiv-mcp`
-
-## Phase 1: Implement
-
-### Step 1: Create `src/mcp/arxiv.ts`
-- Follow static export pattern (same as `context7.ts` and `grep-app.ts`)
-- arXiv API is public, no auth needed
-- URL: `https://mcp.arxiv.org` (hypothetical remote MCP endpoint)
-- If no remote MCP exists for arXiv, this would need to be a stdio MCP or a custom HTTP wrapper. For this plan, we assume a remote MCP endpoint pattern consistent with existing built-ins.
-
-### Step 2: Update `src/mcp/types.ts`
-- Add `"arxiv"` to `McpNameSchema` enum: `z.enum(["websearch", "context7", "grep_app", "arxiv"])`
-
-### Step 3: Update `src/mcp/index.ts`
-- Import `arxiv` from `"./arxiv"`
-- Add conditional block in `createBuiltinMcps()`:
-  ```typescript
-  if (!disabledMcps.includes("arxiv")) {
-    mcps.arxiv = arxiv
-  }
-  ```
-
-### Step 4: Create `src/mcp/arxiv.test.ts`
-- Test arXiv config shape (type, url, enabled, oauth)
-- Follow pattern from existing tests (given/when/then)
-
-### Step 5: Update `src/mcp/index.test.ts`
-- Update expected MCP count from 3 to 4
-- Add `"arxiv"` to `toHaveProperty` checks
-- Add `"arxiv"` to the "all disabled" test case
-
-### Step 6: Update `src/mcp/AGENTS.md`
-- Add arxiv row to the built-in MCPs table
-
-### Step 7: Local validation
-- `bun run typecheck`
-- `bun test src/mcp/`
-- `bun run build`
-
-### Atomic commits (in order):
-1. `feat(mcp): add arxiv paper search built-in MCP` - arxiv.ts + types.ts update
-2. `test(mcp): add arxiv MCP tests` - arxiv.test.ts + index.test.ts updates
-3. `docs(mcp): update AGENTS.md with arxiv MCP` - AGENTS.md update
-
-## Phase 2: PR Creation
-
-1. `git push -u origin feat/arxiv-mcp`
-2. `gh pr create --base dev --title "feat(mcp): add built-in arXiv paper search MCP" --body-file /tmp/pull-request-arxiv-mcp-*.md`
-
-## Phase 3: Verify Loop
-
-### Gate A: CI
-- Wait for `ci.yml` workflow (tests, typecheck, build)
-- `gh run watch` or poll `gh pr checks`
-
-### Gate B: review-work
-- Run `/review-work` skill (5-agent parallel review)
-- All 5 agents must pass: Oracle (goal), Oracle (code quality), Oracle (security), QA execution, context mining
-
-### Gate C: Cubic
-- Wait for cubic-dev-ai[bot] automated review
-- Must show "No issues found"
-- If issues found, fix and re-push
-
-### Failure handling:
-- Gate A fail: fix locally, amend or new commit, re-push
-- Gate B fail: address review-work findings, new commit
-- Gate C fail: address Cubic findings, new commit
-- Re-enter verify loop from Gate A
-
-## Phase 4: Merge
-
-1. `gh pr merge --squash --delete-branch`
-2. `git worktree remove ../omo-wt/feat/arxiv-mcp`
-3. `git branch -D feat/arxiv-mcp` (if not auto-deleted)

package/.opencode/skills/work-with-pr-workspace/iteration-1/eval-4/with_skill/outputs/pr-description.md

@@ -1,51 +0,0 @@
-# PR: feat(mcp): add built-in arXiv paper search MCP
-
-## Title
-
-`feat(mcp): add built-in arXiv paper search MCP`
-
-## Body
-
-```markdown
-## Summary
-
-Closes #100
-
-- Add `arxiv` as 4th built-in remote MCP for arXiv paper search
-- Follows existing static export pattern (same as `grep_app`, `context7`)
-- No auth required, disableable via `disabled_mcps: ["arxiv"]`
-
-## Changes
-
-- `src/mcp/arxiv.ts` - new MCP config (static export, remote type)
-- `src/mcp/types.ts` - add `"arxiv"` to `McpNameSchema` enum
-- `src/mcp/index.ts` - register arxiv in `createBuiltinMcps()`
-- `src/mcp/arxiv.test.ts` - config shape tests
-- `src/mcp/index.test.ts` - update counts, add disable test
-- `src/mcp/AGENTS.md` - document new MCP
-
-## Usage
-
-Enabled by default. Disable with:
-
-```jsonc
-// .opencode/oh-my-opencode.jsonc
-{
-  "disabled_mcps": ["arxiv"]
-}
-```
-
-## Validation
-
-- [x] `bun run typecheck` passes
-- [x] `bun test src/mcp/` passes
-- [x] `bun run build` passes
-```
-
-## Labels
-
-`enhancement`, `mcp`
-
-## Base branch
-
-`dev`

package/.opencode/skills/work-with-pr-workspace/iteration-1/eval-4/with_skill/outputs/verification-strategy.md

@@ -1,69 +0,0 @@
-# Verification Strategy: Issue #100 - arXiv MCP
-
-## Gate A: CI (`ci.yml`)
-
-### What runs
-- `bun test` (split: mock-heavy isolated + batch) - must include new `arxiv.test.ts` and updated `index.test.ts`
-- `bun run typecheck` - validates `McpNameSchema` enum change propagates correctly
-- `bun run build` - ensures no build regressions
-
-### How to monitor
-```bash
-gh pr checks <pr-number> --watch
-```
-
-### Failure scenarios
-| Failure | Likely cause | Fix |
-|---------|-------------|-----|
-| Type error in `types.ts` | Enum value not matching downstream consumers | Check all `McpName` usages via `lsp_find_references` |
-| Test count mismatch in `index.test.ts` | Forgot to update `toHaveLength()` from 3 to 4 | Update all length assertions |
-| Build failure | Import path or barrel export issue | Verify `src/mcp/index.ts` exports are clean |
-
-### Retry
-Fix locally in worktree, new commit, `git push`.
-
-## Gate B: review-work (5-agent)
-
-### Agents and focus areas
-| Agent | What it checks for this PR |
-|-------|--------------------------|
-| Oracle (goal) | Does arxiv MCP satisfy issue #100 requirements? |
-| Oracle (code quality) | Follows `grep-app.ts` pattern? No SRP violations? < 200 LOC? |
-| Oracle (security) | No credentials hardcoded, no auth bypass |
-| QA (execution) | Run tests, verify disable mechanism works |
-| Context (mining) | Check issue #100 for any missed requirements |
-
-### Pass criteria
-All 5 must pass. Any single failure blocks.
-
-### Failure handling
-- Read each agent's report
-- Address findings with new atomic commits
-- Re-run full verify loop from Gate A
-
-## Gate C: Cubic (`cubic-dev-ai[bot]`)
-
-### Expected review scope
-- Config shape consistency across MCPs
-- Test coverage for new MCP
-- Schema type safety
-
-### Pass criteria
-Comment from `cubic-dev-ai[bot]` containing "No issues found".
-
-### Failure handling
-- Read Cubic's specific findings
-- Fix with new commit
-- Re-push, re-enter Gate A
-
-## Pre-merge checklist
-- [ ] Gate A: CI green
-- [ ] Gate B: All 5 review-work agents pass
-- [ ] Gate C: Cubic "No issues found"
-- [ ] No unresolved review comments
-- [ ] PR has at least 1 approval (if required by branch protection)
-
-## Post-merge
-1. `gh pr merge --squash --delete-branch`
-2. `git worktree remove ../omo-wt/feat/arxiv-mcp`
-3. Verify merge commit on `dev` branch

package/.opencode/skills/work-with-pr-workspace/iteration-1/eval-4/with_skill/timing.json

@@ -1 +0,0 @@
-{"total_tokens": null, "duration_ms": 152000, "total_duration_seconds": 152}

package/.opencode/skills/work-with-pr-workspace/iteration-1/eval-4/without_skill/grading.json

@@ -1,10 +0,0 @@
-{
-  "run_id": "eval-4-without_skill",
-  "expectations": [
-    {"text": "Plan uses git worktree in a sibling directory", "passed": true, "evidence": "git worktree add ../omo-arxiv-mcp dev — agent independently chose worktree"},
-    {"text": "New MCP follows existing pattern from src/mcp/", "passed": true, "evidence": "Follows grep-app.ts pattern"},
-    {"text": "Verification loop includes all 3 gates", "passed": false, "evidence": "Only mentions bun test/typecheck/build. No review-work or Cubic."},
-    {"text": "PR targets dev branch", "passed": true, "evidence": "--base dev"},
-    {"text": "Runs local checks before pushing", "passed": true, "evidence": "bun test src/mcp/, bun run typecheck, bun run build"}
-  ]
-}

package/.opencode/skills/work-with-pr-workspace/iteration-1/eval-4/without_skill/outputs/code-changes.md

@@ -1,252 +0,0 @@
-# Code Changes: Built-in arXiv MCP
-
-## 1. NEW FILE: `src/mcp/arxiv.ts`
-
-```typescript
-export const arxiv = {
-  type: "remote" as const,
-  url: "https://mcp.arxiv.org",
-  enabled: true,
-  oauth: false as const,
-}
-```
-
-> **Note:** The URL `https://mcp.arxiv.org` is a placeholder. The actual endpoint needs to be verified. If no hosted arXiv MCP exists, alternatives include community-hosted servers or a self-hosted wrapper around the arXiv REST API (`export.arxiv.org/api/query`). This would be the single blocker requiring resolution before merging.
-
-Pattern followed: `grep-app.ts` (static export, no auth, no config factory needed since arXiv API is public).
-
----
-
-## 2. MODIFY: `src/mcp/types.ts`
-
-```diff
- import { z } from "zod"
-
--export const McpNameSchema = z.enum(["websearch", "context7", "grep_app"])
-+export const McpNameSchema = z.enum(["websearch", "context7", "grep_app", "arxiv"])
-
- export type McpName = z.infer<typeof McpNameSchema>
-
- export const AnyMcpNameSchema = z.string().min(1)
-
- export type AnyMcpName = z.infer<typeof AnyMcpNameSchema>
-```
-
----
-
-## 3. MODIFY: `src/mcp/index.ts`
-
-```diff
- import { createWebsearchConfig } from "./websearch"
- import { context7 } from "./context7"
- import { grep_app } from "./grep-app"
-+import { arxiv } from "./arxiv"
- import type { OhMyOpenCodeConfig } from "../config/schema"
-
--export { McpNameSchema, type McpName } from "./types"
-+export { McpNameSchema, type McpName } from "./types"
-
- type RemoteMcpConfig = {
-   type: "remote"
-   url: string
-   enabled: boolean
-   headers?: Record<string, string>
-   oauth?: false
- }
-
- export function createBuiltinMcps(disabledMcps: string[] = [], config?: OhMyOpenCodeConfig) {
-   const mcps: Record<string, RemoteMcpConfig> = {}
-
-   if (!disabledMcps.includes("websearch")) {
-     mcps.websearch = createWebsearchConfig(config?.websearch)
-   }
-
-   if (!disabledMcps.includes("context7")) {
-     mcps.context7 = context7
-   }
-
-   if (!disabledMcps.includes("grep_app")) {
-     mcps.grep_app = grep_app
-   }
-
-+  if (!disabledMcps.includes("arxiv")) {
-+    mcps.arxiv = arxiv
-+  }
-+
-   return mcps
- }
-```
-
----
-
-## 4. MODIFY: `src/mcp/index.test.ts`
-
-Changes needed in existing tests (count 3 → 4) plus one new test:
-
-```diff
- describe("createBuiltinMcps", () => {
-   test("should return all MCPs when disabled_mcps is empty", () => {
-     // given
-     const disabledMcps: string[] = []
-
-     // when
-     const result = createBuiltinMcps(disabledMcps)
-
-     // then
-     expect(result).toHaveProperty("websearch")
-     expect(result).toHaveProperty("context7")
-     expect(result).toHaveProperty("grep_app")
--    expect(Object.keys(result)).toHaveLength(3)
-+    expect(result).toHaveProperty("arxiv")
-+    expect(Object.keys(result)).toHaveLength(4)
-   })
-
-   test("should filter out disabled built-in MCPs", () => {
-     // given
-     const disabledMcps = ["context7"]
-
-     // when
-     const result = createBuiltinMcps(disabledMcps)
-
-     // then
-     expect(result).toHaveProperty("websearch")
-     expect(result).not.toHaveProperty("context7")
-     expect(result).toHaveProperty("grep_app")
--    expect(Object.keys(result)).toHaveLength(2)
-+    expect(result).toHaveProperty("arxiv")
-+    expect(Object.keys(result)).toHaveLength(3)
-   })
-
-   test("should filter out all built-in MCPs when all disabled", () => {
-     // given
--    const disabledMcps = ["websearch", "context7", "grep_app"]
-+    const disabledMcps = ["websearch", "context7", "grep_app", "arxiv"]
-
-     // when
-     const result = createBuiltinMcps(disabledMcps)
-
-     // then
-     expect(result).not.toHaveProperty("websearch")
-     expect(result).not.toHaveProperty("context7")
-     expect(result).not.toHaveProperty("grep_app")
-+    expect(result).not.toHaveProperty("arxiv")
-     expect(Object.keys(result)).toHaveLength(0)
-   })
-
-   test("should ignore custom MCP names in disabled_mcps", () => {
-     // given
-     const disabledMcps = ["context7", "playwright", "custom"]
-
-     // when
-     const result = createBuiltinMcps(disabledMcps)
-
-     // then
-     expect(result).toHaveProperty("websearch")
-     expect(result).not.toHaveProperty("context7")
-     expect(result).toHaveProperty("grep_app")
--    expect(Object.keys(result)).toHaveLength(2)
-+    expect(result).toHaveProperty("arxiv")
-+    expect(Object.keys(result)).toHaveLength(3)
-   })
-
-   test("should handle empty disabled_mcps by default", () => {
-     // given
-     // when
-     const result = createBuiltinMcps()
-
-     // then
-     expect(result).toHaveProperty("websearch")
-     expect(result).toHaveProperty("context7")
-     expect(result).toHaveProperty("grep_app")
--    expect(Object.keys(result)).toHaveLength(3)
-+    expect(result).toHaveProperty("arxiv")
-+    expect(Object.keys(result)).toHaveLength(4)
-   })
-
-   test("should only filter built-in MCPs, ignoring unknown names", () => {
-     // given
-     const disabledMcps = ["playwright", "sqlite", "unknown-mcp"]
-
-     // when
-     const result = createBuiltinMcps(disabledMcps)
-
-     // then
-     expect(result).toHaveProperty("websearch")
-     expect(result).toHaveProperty("context7")
-     expect(result).toHaveProperty("grep_app")
--    expect(Object.keys(result)).toHaveLength(3)
-+    expect(result).toHaveProperty("arxiv")
-+    expect(Object.keys(result)).toHaveLength(4)
-   })
-
-+  test("should filter out arxiv when disabled", () => {
-+    // given
-+    const disabledMcps = ["arxiv"]
-+
-+    // when
-+    const result = createBuiltinMcps(disabledMcps)
-+
-+    // then
-+    expect(result).toHaveProperty("websearch")
-+    expect(result).toHaveProperty("context7")
-+    expect(result).toHaveProperty("grep_app")
-+    expect(result).not.toHaveProperty("arxiv")
-+    expect(Object.keys(result)).toHaveLength(3)
-+  })
-+
-   // ... existing tavily test unchanged
- })
-```
-
----
-
-## 5. MODIFY: `src/mcp/AGENTS.md`
-
-```diff
--# src/mcp/ — 3 Built-in Remote MCPs
-+# src/mcp/ — 4 Built-in Remote MCPs
-
- **Generated:** 2026-03-06
-
- ## OVERVIEW
-
--Tier 1 of the three-tier MCP system. 3 remote HTTP MCPs created via `createBuiltinMcps(disabledMcps, config)`.
-+Tier 1 of the three-tier MCP system. 4 remote HTTP MCPs created via `createBuiltinMcps(disabledMcps, config)`.
-
- ## BUILT-IN MCPs
-
- | Name | URL | Env Vars | Tools |
- |------|-----|----------|-------|
- | **websearch** | `mcp.exa.ai` (default) or `mcp.tavily.com` | `EXA_API_KEY` (optional), `TAVILY_API_KEY` (if tavily) | Web search |
- | **context7** | `mcp.context7.com/mcp` | `CONTEXT7_API_KEY` (optional) | Library documentation |
- | **grep_app** | `mcp.grep.app` | None | GitHub code search |
-+| **arxiv** | `mcp.arxiv.org` | None | arXiv paper search |
-
- ...
-
- ## FILES
-
- | File | Purpose |
- |------|---------|
- | `index.ts` | `createBuiltinMcps()` factory |
--| `types.ts` | `McpNameSchema`: "websearch" \| "context7" \| "grep_app" |
-+| `types.ts` | `McpNameSchema`: "websearch" \| "context7" \| "grep_app" \| "arxiv" |
- | `websearch.ts` | Exa/Tavily provider with config |
- | `context7.ts` | Context7 with optional auth header |
- | `grep-app.ts` | Grep.app (no auth) |
-+| `arxiv.ts` | arXiv paper search (no auth) |
-```
-
----
-
-## Summary of Touched Files
-
-| File | Lines Changed | Type |
-|------|--------------|------|
-| `src/mcp/arxiv.ts` | +6 (new) | Create |
-| `src/mcp/types.ts` | 1 line modified | Modify |
-| `src/mcp/index.ts` | +5 (import + block) | Modify |
-| `src/mcp/index.test.ts` | ~20 lines (count fixes + new test) | Modify |
-| `src/mcp/AGENTS.md` | ~6 lines | Modify |
-
-Total: ~37 lines added/modified across 5 files. Minimal, surgical change.

package/.opencode/skills/work-with-pr-workspace/iteration-1/eval-4/without_skill/outputs/execution-plan.md

@@ -1,83 +0,0 @@
-# Execution Plan: Add Built-in arXiv MCP (Issue #100)
-
-## Pre-Implementation
-
-1. **Create worktree + branch**
-   ```bash
-   git worktree add ../omo-arxiv-mcp dev
-   cd ../omo-arxiv-mcp
-   git checkout -b feat/arxiv-mcp
-   ```
-
-2. **Verify arXiv MCP endpoint exists**
-   - The arXiv API is public (`export.arxiv.org/api/query`) but has no native MCP endpoint
-   - Need to identify a hosted remote MCP server for arXiv (e.g., community-maintained or self-hosted)
-   - If no hosted endpoint exists, consider alternatives: (a) use a community-hosted one from the MCP registry, (b) flag this in the PR and propose a follow-up for hosting
-   - For this plan, assume a remote MCP endpoint at a URL like `https://mcp.arxiv.org` or a third-party equivalent
-
-## Implementation Steps (4 files to modify, 2 files to create)
-
-### Step 1: Create `src/mcp/arxiv.ts`
-- Follow the `grep-app.ts` pattern (simplest: static export, no auth, no config)
-- arXiv API is public, so no API key needed
-- Export a `const arxiv` with `type: "remote"`, `url`, `enabled: true`, `oauth: false`
-
-### Step 2: Update `src/mcp/types.ts`
-- Add `"arxiv"` to the `McpNameSchema` z.enum array
-- This makes it a recognized built-in MCP name
-
-### Step 3: Update `src/mcp/index.ts`
-- Import `arxiv` from `"./arxiv"`
-- Add the `if (!disabledMcps.includes("arxiv"))` block inside `createBuiltinMcps()`
-- Place it after `grep_app` block (alphabetical among new additions, or last)
-
-### Step 4: Update `src/mcp/index.test.ts`
-- Update test "should return all MCPs when disabled_mcps is empty" to expect 4 MCPs instead of 3
-- Update test "should filter out all built-in MCPs when all disabled" to include "arxiv" in the disabled list and expect it not present
-- Update test "should handle empty disabled_mcps by default" to expect 4 MCPs
-- Update test "should only filter built-in MCPs, ignoring unknown names" to expect 4 MCPs
-- Add new test: "should filter out arxiv when disabled"
-
-### Step 5: Create `src/mcp/arxiv.test.ts` (optional, only if factory pattern used)
-- If using static export (like grep-app), no separate test file needed
-- If using factory with config, add tests following `websearch.test.ts` pattern
-
-### Step 6: Update `src/mcp/AGENTS.md`
-- Add arxiv to the built-in MCPs table
-- Update "3 Built-in Remote MCPs" to "4 Built-in Remote MCPs"
-- Add arxiv to the FILES table
-
-## Post-Implementation
-
-### Verification
-```bash
-bun test src/mcp/         # Run MCP tests
-bun run typecheck          # Verify no type errors
-bun run build             # Verify build passes
-```
-
-### PR Creation
-```bash
-git add src/mcp/arxiv.ts src/mcp/types.ts src/mcp/index.ts src/mcp/index.test.ts src/mcp/AGENTS.md
-git commit -m "feat(mcp): add built-in arxiv paper search MCP"
-git push -u origin feat/arxiv-mcp
-gh pr create --title "feat(mcp): add built-in arxiv paper search MCP" --body-file /tmp/pull-request-arxiv-mcp-....md --base dev
-```
-
-## Risk Assessment
-
-| Risk | Likelihood | Mitigation |
-|------|-----------|------------|
-| No hosted arXiv MCP endpoint exists | Medium | Research MCP registries; worst case, create a minimal hosted wrapper or use a community server |
-| Existing tests break due to MCP count change | Low | Update hardcoded count assertions from 3 to 4 |
-| Config schema needs updates | None | `disabled_mcps` uses `AnyMcpNameSchema` (any string), not `McpNameSchema`, so no schema change needed for disable functionality |
-
-## Files Changed Summary
-
-| File | Action | Description |
-|------|--------|-------------|
-| `src/mcp/arxiv.ts` | Create | Static remote MCP config export |
-| `src/mcp/types.ts` | Modify | Add "arxiv" to McpNameSchema enum |
-| `src/mcp/index.ts` | Modify | Import + register in createBuiltinMcps() |
-| `src/mcp/index.test.ts` | Modify | Update count assertions, add arxiv-specific test |
-| `src/mcp/AGENTS.md` | Modify | Update docs to reflect 4 MCPs |