Repository navigation
feat(tools): computed price analytics — statistics, trend, correlation, spread (#195) - #199
Closed
karlwaldman wants to merge 2 commits into
Closed
karlwaldman wants to merge 2 commits into
karlwaldman wants to merge 2 commits into
Conversation
Exposes the four /v1/analytics operations ledger v8 deferred, verified live on 2026-11-10 with the Scale smoke key: - opa_get_price_statistics: GET /v1/analytics/statistics (z-score, percentile, annualized volatility). - opa_get_price_trend: GET /v1/analytics/trend (analysis, sma, ema, rsi, levels). - opa_get_price_correlation: GET /v1/analytics/correlation (analysis, rolling, matrix of 2-10 codes). - opa_get_spread_statistics: GET /v1/analytics/spread (list, current, historical). Every description states the Professional/Scale gate. The 403 sample_data is never rendered, and the body's `tier` field is not rendered because it reads "free" for Gen-2 paid plans. Series capped at 400 rows. Ledger v9 (analytics family exposed), live-contract catalog entries, README rows, tool counts 44/40. 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
Analytics: the four
/v1/analyticsoperations ledger v8 deferred, verified live on 2026-11-10 with the Scale smoke key. Theanalyticsfamily is now exposed.opa_get_price_statisticsopa_get_price_trendopa_get_price_correlationopa_get_spread_statisticsTIER_FEATURESon oilpriceapi-api main.sample_datais never rendered as data (mocked test).tierfield is not rendered: it reads"free"for the Scale account. Filed oilpriceapi-api#10761 (confirmed-live).analyticslive-smoke shape was added. Tool counts 40 -> 44 (read 36 -> 40).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