chore(mcp): remove ENABLE_OOXML_TOOLS feature flag

The flag existed so the new structural tools could land without
affecting api.ooxml.dev/mcp's surface until prod was ready. The
operational plan is to populate the prod schema graph before merging,
at which point the flag is just friction. Drops:

  - ENABLE_OOXML_TOOLS env var on Env / OoxmlEnv
  - ooxmlToolsEnabled() and the gating in tools/list and tools/call
  - The defensive 'method-not-found while flag is off' branch

Both tools/list and tools/call now expose the OOXML tools
unconditionally. Worker bundle builds clean; 44 / 0 tests still pass.

Note: prod populate must run before this merges. The current per-row
INSERT pattern is slow against Neon over public internet
(~10-20 minutes for the WML closure); batching is the next step
operationally.

Author: caiopizzol

CLAUDE.md

@@ -105,15 +105,15 @@ The XML you provide is wrapped in a minimal `w:document > w:body` structure auto
 
 ## MCP Server
 
-Cloudflare Worker exposing two flavors of MCP tools backed by the same database:
+Cloudflare Worker exposing two flavors of MCP tools backed by the same database.
 
-Always-on (semantic search over the spec PDF):
+Semantic search over the spec PDF (powered by `spec_content`):
 
 - `search_ecma_spec` - semantic vector search across 18,000+ spec chunks
 - `get_section` - fetch a specific section by ID (e.g., "17.3.1.24")
 - `list_parts` - browse the spec structure
 
-Behind `ENABLE_OOXML_TOOLS` (structural queries over the XSD schema graph):
+Structural queries over the XSD schema graph (powered by `xsd_*` tables):
 
 - `ooxml_lookup_element` / `ooxml_lookup_type` - canonical symbol info
 - `ooxml_children` - legal children of an element/type/group, in document order

apps/mcp-server/src/index.ts

@@ -16,13 +16,6 @@ import { handleMcpRequest } from "./mcp";
 export interface Env {
 	DATABASE_URL: string;
 	VOYAGE_API_KEY: string;
-	/**
-	 * Feature flag for the OOXML structural tools. Set to "true" to expose
-	 * ooxml_lookup_element / ooxml_lookup_type / ooxml_children /
-	 * ooxml_attributes / ooxml_enum / ooxml_namespace_info via tools/list
-	 * and tools/call. Default off.
-	 */
-	ENABLE_OOXML_TOOLS?: string;
 }
 
 // Part descriptions

apps/mcp-server/src/mcp.ts

@@ -7,7 +7,7 @@
 import { createDb } from "./db";
 import { embedQuery } from "./embeddings";
 import type { Env } from "./index";
-import { callOoxmlTool, isOoxmlTool, OOXML_TOOL_DEFS, ooxmlToolsEnabled } from "./ooxml-tools";
+import { callOoxmlTool, isOoxmlTool, OOXML_TOOL_DEFS } from "./ooxml-tools";
 
 // JSON-RPC types
 interface JsonRpcRequest {
@@ -133,12 +133,11 @@ function handleInitialize(id: number | string | null): JsonRpcResponse {
 	};
 }
 
-function handleToolsList(id: number | string | null, env: Env): JsonRpcResponse {
-	const tools = ooxmlToolsEnabled(env) ? [...TOOLS, ...OOXML_TOOL_DEFS] : TOOLS;
+function handleToolsList(id: number | string | null): JsonRpcResponse {
 	return {
 		jsonrpc: "2.0",
 		id,
-		result: { tools },
+		result: { tools: [...TOOLS, ...OOXML_TOOL_DEFS] },
 	};
 }
 
@@ -162,17 +161,9 @@ async function handleToolsCall(
 	try {
 		let resultText: string;
 
-		// OOXML tools are feature-flagged; tools/list filters them out when the flag
-		// is off, so callers should not see these tool names. Defensive check here in
-		// case a caller hand-crafts a request.
+		// Structural OOXML tools share the dispatch with the existing semantic
+		// tools below.
 		if (isOoxmlTool(name)) {
-			if (!ooxmlToolsEnabled(env)) {
-				return {
-					jsonrpc: "2.0",
-					id,
-					error: { code: METHOD_NOT_FOUND, message: `Unknown tool: ${name}` },
-				};
-			}
 			resultText = await callOoxmlTool(name, args ?? {}, env);
 			return {
 				jsonrpc: "2.0",
@@ -393,7 +384,7 @@ export async function handleMcpRequest(request: Request, env: Env): Promise<Resp
 			return new Response(null, { status: 202 });
 
 		case "tools/list":
-			return jsonResponse(handleToolsList(id, env));
+			return jsonResponse(handleToolsList(id));
 
 		case "tools/call":
 			return jsonResponse(await handleToolsCall(id, body.params, env));

apps/mcp-server/src/ooxml-tools.ts

@@ -1,7 +1,5 @@
 /**
- * Read-only structural MCP tools backed by the OOXML schema graph. Gated by
- * ENABLE_OOXML_TOOLS, which filters both tools/list discovery and tools/call
- * dispatch so the public surface stays unchanged until the flag is set.
+ * Read-only structural MCP tools backed by the OOXML schema graph.
  *
  * Tools:
  *   ooxml_lookup_element, ooxml_lookup_type, ooxml_children,
@@ -33,12 +31,6 @@ export const DEFAULT_PROFILE = "transitional";
 
 export interface OoxmlEnv {
 	DATABASE_URL: string;
-	ENABLE_OOXML_TOOLS?: string;
-}
-
-export function ooxmlToolsEnabled(env: OoxmlEnv): boolean {
-	const v = env.ENABLE_OOXML_TOOLS;
-	return v === "true" || v === "1";
 }
 
 export const OOXML_TOOL_DEFS = [
@@ -215,13 +207,7 @@ export async function runOoxmlTool(
 				} else if (!elementSym) {
 					// Fall back to looking for a named xsd:group with this qname (so
 					// EG_PContent and friends are reachable directly).
-					typeSym = await lookupSymbol(
-						sql,
-						q.qname.namespace,
-						q.qname.localName,
-						"group",
-						profile,
-					);
+					typeSym = await lookupSymbol(sql, q.qname.namespace, q.qname.localName, "group", profile);
 				}
 			}
 			if (!typeSym) {