feat(docs): add direct table operations#729
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 10, 2026, 6:53 AM ET / 10:53 UTC. Summary Reproducibility: not applicable. as a bug reproduction: this is a feature PR. Current main inspection shows the top-level table commands are absent, while the PR diff adds them and reports live Google Docs proof. Review metrics: 3 noteworthy metrics.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Risk before merge
Maintainer options:
Next step before merge
Security Review detailsBest possible solution: Land after maintainers accept the direct command contract and the fail-closed Do we have a high-confidence way to reproduce the issue? Not applicable as a bug reproduction: this is a feature PR. Current main inspection shows the top-level table commands are absent, while the PR diff adds them and reports live Google Docs proof. Is this the best way to solve the issue? Yes, with a maintainer acceptance caveat: sharing request builders between sed and direct commands is the narrow maintainable direction, but the stricter sed behavior is a product/compatibility choice before merge. AGENTS.md: found and applied where relevant. Codex review notes: model gpt-5.5, reasoning high; reviewed against e1be268f70a6. 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
text:disambiguation,*, and tab selectiondocs sed, remove duplicate mutation paths, and preserve revision control across capped batchesProof
DEVELOPER_DIR=/Library/Developer/CommandLineTools make ci[email protected]:Closes #686