docs(design): strike what the sheet editing plan already shipped - #889
Merged
Merged
Conversation
Steps 0 to 3 landed, and the plan still read as though they had not. Every low-hanging fruit is in, so the section says so in three lines instead of twelve; five complications and two spikes are marked with what answered them. `odr::ErrorCode` answered the last open question: it is the one table the page and every binding report a refusal from. What is left is step 4 and step 5, and the four notes that are still true: the cut, the log across views, encrypted packages, and the locale. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VVmjmddv2Ui17Nptc1ggue
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.
🤖 Generated with Claude Code
Docs only.
docs/design/spreadsheet-editing.mdsays of itself that it is "a plan, not a record — update it as steps land", and steps 0 to 3 landed (#884 → #888) without three of its sections following.What changed
.odscached), the third by step 2's run-index insert.odr::ErrorCode(feat!: oneErrorCodespace every binding reports #883) is the one C++ table the page and every binding report a refusal from, written into the script byfrontend.cpp::write_error_codes.What is left, after this
Step 4 (the evaluator, the incremental recompute,
Document::recalculate, formula input) and step 5.src/odr/internal/formula/holds a parser, a writer, an AST and the dependency walk, and no evaluator.https://claude.ai/code/session_01VVmjmddv2Ui17Nptc1ggue