Repository navigation
feat(tools): fuel surcharge calculator, price alert dry run, service status (#195) - #197
Closed
karlwaldman wants to merge 2 commits into
Closed
karlwaldman wants to merge 2 commits into
karlwaldman wants to merge 2 commits into
Conversation
…status (#195) Exposes three operations ledger v8 deferred, each verified live on 2026-11-10: - opa_calculate_fuel_surcharge: GET /v1/fuel-surcharge/calculate. Percent or per-mile schedule on the EIA weekly diesel index, seven regions, 1-52 weeks. Checks per-method required inputs before spending a request and states when the API marks the index stale. - opa_test_price_alert: POST /v1/alerts/{alert_id}/test. Read scope, automation category; a dry run that sends nothing, described as metered. Covered in the live price-alert lifecycle on the synthetic alert. - opa_get_service_status: GET /v1/status. Keyless, core category, entitlement none. A 503 (stale snapshot) is reported as health unknown and not retried. Ledger v9 (families fuel-surcharge, price-alerts, data-quality now exposed), live-contract catalog entries, README rows, tool counts 43/39. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01B3sTPV81755NVtvVwb9Cd8
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
 |
karlwaldman
added
the
risk:elevated
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
Member
Author
|
Superseded by #200. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Tools for three operations ledger v8 deferred, each verified live on 2026-11-10.
opa_calculate_fuel_surchargeopa_test_price_alertopa_get_service_statusstale.readOnlyHint. The API sends nothing and changes nothing. The description says the call is metered. Ids are numeric (live alert id 134), so the schema accepts digits only. The live price-alert lifecycle now dry-runs the synthetic alert through the real handler.entitlement: none. A 503 (snapshot missing or 15 or more minutes old) is reported as health unknown and is not retried.include_commoditieslists only series that are not healthy or not current.fuel-surcharge,price-alertsanddata-qualityare now fully exposed. Tool counts 40 -> 43 (read 36 -> 39). The core profile gainsopa_get_service_status.opa_delete_price_alert's schema text calls alert ids UUIDs, but the API returns integers.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 onto124c1de(fix(ci): unblock main - npm audit fix (qs), ledger v8 for 18 new API operations, relative test date #193).npm run test:livewith the production smoke key (Scale) andSMOKE_WRITE_CONTRACTS=1: every contract PASSED, including the new tools; 0 failed, 0 cannot-check.Merge notes
ledgerVersion8 -> 9 and the hardcoded tool counts. Whichever merges second needs a mechanical rebase: nextledgerVersion, summed counts.pai-pr-risk --armcannot arm on this repo until main has no required status checks; pai-pr-risk --arm cannot arm any PR #194 (no required status checks on main) is fixed.Refs #195.
🤖 Generated with Claude Code
https://claude.ai/code/session_01B3sTPV81755NVtvVwb9Cd8