Skip to content

Fix finance runtime contract and validation inconsistencies - #29

Merged
id774 merged 1 commit into
masterfrom
claude/finance-doc-bug-fixes-q3daoo
Sep 9, 2026
Merged

id774 merged 1 commit into
masterfrom
claude/finance-doc-bug-fixes-q3daoo

Conversation

@id774

@id774 id774 commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Keep chart-only runs on stored prices and make missing stored data fail
    without fetching.
  • Persist only trading-day price rows, dropping all-empty calendar gaps.
  • Reject malformed configuration values before work starts.
  • Reject non-object J-Quants data rows instead of silently discarding them.
  • Align the related CLI, data-contract and deployment documentation.

Compatibility

  • CLI option names and valid option semantics are unchanged.
  • Valid configuration values and precedence are unchanged.
  • Generated file names, separators, columns and column order are unchanged.
  • Indicator, model and summary formulas are unchanged.
  • The corrected behaviours were accidental fallback/acceptance paths, not
    supported compatibility paths.

Version

  • Candidate repository version: v1.0.2
  • Release Date: TBD

Validation

  • pytest — PASS (490 passed, 2 deselected)
  • ruff check . — PASS
  • sh -n run.sh — PASS
  • sh -n deploy.sh — PASS

Scope

No dependency, cron, formula, provider endpoint, or finance-dashboard change.

🤖 Generated with Claude Code

https://claude.ai/code/session_01BGJm1r7JvjV9P8PCuuCTEr


Generated by Claude Code

@id774
id774 merged commit 533b72d into master Sep 9, 2026
3 checks passed
@id774
id774 deleted the claude/finance-doc-bug-fixes-q3daoo branch September 9, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants