Skip to content

feat(tools): well permit list and any-key permit preview (#195) - #198

Closed
karlwaldman wants to merge 2 commits into
mainfrom
feat/mcp-drilling-tools-1110
Closed

karlwaldman wants to merge 2 commits into
mainfrom
feat/mcp-drilling-tools-1110

Conversation

@karlwaldman

Copy link
Copy Markdown
Member

Summary

Drilling: the two well-permit operations ledger v8 deferred, verified live on 2026-11-10.

Tool Operation Gate
opa_list_well_permits GET /v1/ei/well-permits Well Permits subscription (add-on or Scale); 403 otherwise
opa_preview_well_permits GET /v1/ei/well-permits/preview any key, including Free; at most 25 rows, one state, 90 days
  • The list tool takes state names or codes (resolved and deduped), a permit-date range, page, and per_page 1-100. It renders a table and offers the next page.
  • The preview tool renders as_of, data age, the stale flag and per-state date coverage, and points at the subscription tools for full history.
  • Regulator rows with absent fields render as -, never null.
  • Ledger v9: two rows flipped to exposed (family well-lifecycle stays partial). Tool counts 40 -> 42 (read 36 -> 38).

Verification (local, 2026-11-10, rebased on main 124c1de)

  • npm test: all suites pass (see per-PR counts in the commit).
  • npm run build, npm run check:readme-tools, npm run smoke:live-contract-matrix, npm run smoke:package: pass.
  • npm run check:capability-ledger: OK v9 against the live OpenAPI contract (136 operations), rerun after rebasing onto 124c1de (fix(ci): unblock main - npm audit fix (qs), ledger v8 for 18 new API operations, relative test date #193).
  • npm run test:live with the production smoke key (Scale) and SMOKE_WRITE_CONTRACTS=1: every contract PASSED, including the new tools; 0 failed, 0 cannot-check.
  • Plan-gated 403 paths were not observable live: both available test keys are on the same Scale account. They are covered by mocked tests against the documented 403 bodies.

Merge notes

Refs #195.

🤖 Generated with Claude Code

https://claude.ai/code/session_01B3sTPV81755NVtvVwb9Cd8

Exposes the two well-permit operations ledger v8 deferred, verified live
on 2026-11-10:

- opa_list_well_permits: GET /v1/ei/well-permits. States (names or
  codes, deduped), permit-date range, page, per_page 1-100. Well Permits
  subscription per the API contract; 403 otherwise.
- opa_preview_well_permits: GET /v1/ei/well-permits/preview. One state,
  1-90 days, at most 25 rows, any key including Free. Renders as_of, data
  age, the stale flag and per-state date coverage.

Ledger v9, live-contract catalog entries, README rows, tool counts 42/38.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B3sTPV81755NVtvVwb9Cd8
@coderabbitai

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 4b040510-f66e-40c1-8d1b-0988e0c91775
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

The public-claims smoke bans hard-coded plan names in source and packed
surfaces, so plan gates are stated as entitlement gates (HTTP 403).
smoke:docker asserts the container's default read inventory, which still
read 36.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B3sTPV81755NVtvVwb9Cd8
@karlwaldman

Copy link
Copy Markdown
Member Author

Superseded by #200.

@karlwaldman
karlwaldman deleted the feat/mcp-drilling-tools-1110 branch November 10, 2026 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:elevated Elevated: auto-merges after an adversarial review APPROVE

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant