Skip to content

fix: preserve formula references across parsing and edits - #1065

Merged
andiwand merged 2 commits into
mainfrom
review/47-formula-references
Oct 5, 2026
Merged

andiwand merged 2 commits into
mainfrom
review/47-formula-references

Conversation

@andiwand

@andiwand andiwand commented Oct 4, 2026 •

Copy link
Copy Markdown
Member

🤖 Generated with Claude Code

Recognize bare whole-row ranges such as SUM(3:5) as references, so evaluation and row edits work. Preserve the exact quoting of OOXML sheet spans when rewriting formulas, including escaped apostrophes.

Ranges whose endpoints name different sheets or documents now report incomplete dependencies instead of claiming a single-sheet rectangle is complete. Check relative shifts before addition so extreme offsets produce #REF! without signed overflow.

Validation: rebuilt odr_test; all 135 formula parser/writer/evaluator, dependency, recalculation and ODF geometry tests passed. Existing tests cover the new row-range, round-trip and shift cases; one integration test checks unresolved cross-sheet ranges and case-insensitive same-sheet endpoints.

@andiwand
andiwand force-pushed the review/46-shared-container-bounds branch from 3d038d9 to 6be6d26 Compare October 5, 2026 14:07
Base automatically changed from review/46-shared-container-bounds to main October 5, 2026 14:30
@andiwand
andiwand force-pushed the review/47-formula-references branch from e0b85f1 to 599d21f Compare October 5, 2026 14:34
@andiwand
andiwand merged commit b760150 into main Oct 5, 2026
24 of 25 checks passed
@andiwand
andiwand deleted the review/47-formula-references branch October 5, 2026 14:47
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