feat(docs): add named range commands#727
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 10, 2026, 2:34 AM ET / 06:34 UTC. Summary Reproducibility: not applicable. as a bug reproduction; this PR implements a new command surface. Source inspection confirms current main lacks the top-level Docs named-range commands, while the branch adds the requested implementation and tests. Review metrics: 1 noteworthy metric.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Next step before merge
Security Review detailsBest possible solution: Keep the PR open for normal maintainer review and merge once repository checks are acceptable. Do we have a high-confidence way to reproduce the issue? Not applicable as a bug reproduction; this PR implements a new command surface. Source inspection confirms current main lacks the top-level Docs named-range commands, while the branch adds the requested implementation and tests. Is this the best way to solve the issue? Yes. The PR uses existing Docs tab-loading, AGENTS.md: found and applied where relevant. Codex review notes: model gpt-5.5, reasoning high; reviewed against e51f6be9dc38. Label changesLabel changes:
Label justifications:
Evidence reviewedWhat I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
Summary
docs named-rangecreate/list/delete/replace commands--atresolutionReplacement is plain text only. Complex-content or Markdown fallback is intentionally out of scope.
Verification
DEVELOPER_DIR=/Library/Developer/CommandLineTools make ci[email protected]:Fixes #692.