Skip to content

docs: consolidate chart and location research - #54

Merged
jacksteamdev merged 2 commits into
mainfrom
docs/consolidated-research
Nov 13, 2026
Merged

jacksteamdev merged 2 commits into
mainfrom
docs/consolidated-research

Conversation

@jacksteamdev

Copy link
Copy Markdown
Contributor

Summary

Preserve three dated research investigations in one archive, with an index identifying their original commits and historical status.

docs/research/
├── README.md                         # dates, provenance, and interpretation
├── ordinary-chart-initialization.md  # calculation and host adapter contract
├── timezone-city-mapping.md          # mapping evidence and unresolved policies
└── lahiri-integration.md             # engine prerequisite investigation

Evidence

  • Before: three research documents existed only on separate branches, absent from origin/main.
    After: all three are present; comparison against their original Git blobs confirms unchanged content apart from Markdown whitespace/table formatting.
  • oxfmt --check docs/research and git diff --cached --check pass. Documentation only; no runtime tests run. Historical external claims were not revalidated.

Merge Danger

Door: two-way

Documentation can be reverted without changing runtime behavior.

Blast Radius: documentation

The index explicitly marks historical findings; engine and product acceptance still require current verification.

@github-actions
github-actions Bot temporarily deployed to pr-54 November 13, 2026 17:43 Destroyed

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs-only change (four research notes). One minor finding inline; nothing affecting runtime behavior.

Comment thread docs/research/lahiri-integration.md Resolved
@github-actions

github-actions Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Review complete. Run log

Scope: incremental since 77cf7a011bb8e0c69280341cfdb61df66b6fdba3; commit: bb07c90a4a66e517ac187ceb1329b3e2dfd10dee.

@github-actions
github-actions Bot temporarily deployed to pr-54 November 13, 2026 20:42 Destroyed

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incremental review of 77cf7a0..bb07c90: docs-only change that removes a machine-specific path from the research notes and updates the README note. No correctness or security issues found.

@jacksteamdev
jacksteamdev merged commit 4f4348a into main Nov 13, 2026
4 checks passed
@jacksteamdev
jacksteamdev deleted the docs/consolidated-research branch November 13, 2026 20:44

This branch was successfully deployed

No deployments
pr-54 — bb07c90a Deployed Nov 13, 2026 by github-actions[bot]
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.

1 participant