feat(server): add result-side tripwires for query_sql and /sql

[rvagg]

adds a notices[] to the results so we can provide warnings when detecting problematic queries that an agent might need an education about

[rvagg]

draft for now, sitting on top of #102, I had to get this off my machine

[Copilot]

🟡 Changes recommended

The current /1e18 heuristic can emit misleading per-token warnings for legitimate non-fp queries (e.g., tx_meta gas calculations), and the system-context wording has a concrete self-contradiction.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds result-side “tripwire” notices for query_sql (MCP) and /sql (REST) responses to help surface common correctness traps in analytical SQL, especially around FilecoinPay totals and per-token decimals.

Changes:

• Introduces sqlNotices(sql, result) heuristics to generate warning strings based on SQL text + result shape.
• Returns notices from both /sql and MCP query_sql when warnings are detected.
• Expands system context guidance and adds Vitest coverage for the new tripwires.

File summaries

File	Description
shared/system-context.ts	Updates SQL guidance for FilecoinPay metrics (two-channel totals, per-token scaling).
server/src/tripwires.ts	Adds sqlNotices implementation that inspects SQL text/results to emit warnings.
server/src/routes.ts	Attaches notices to REST /sql responses.
server/src/mcp-handler.ts	Attaches notices to MCP query_sql tool results and updates tool description text.
server/test/tripwires.test.ts	Adds new Vitest suite covering the tripwire heuristics.

Review details

• Files reviewed: 5/5 changed files
• Comments generated: 3
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.