Skip to content

Remove path-table mutation waiver - #1354

Merged
thekevinbot merged 8 commits into
mainfrom
claude/1330-vtab-stat-cell
Oct 6, 2026
Merged

thekevinbot merged 8 commits into
mainfrom
claude/1330-vtab-stat-cell

Conversation

@thekevinbot

@thekevinbot thekevinbot commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Refs #1330

Remove the path-table stat and cell-writing mutation exemption by injecting scan/stat and content-read effects, then test row construction, lazy content reads, and cell writes. A SQLite-backed unit test also checks that the real context adapter stores the stat cell.

E2E Verification

  • Python e2e receipt pending: the full unfiltered suite will be run on this updated branch now that Test one-level glob listings at speed of light #1317 has merged; a receipt will be added only after it passes.
  • TypeScript SDK e2e: full suite passed (18 tests); receipt retained.
  • Rust unit tests: vtab unit tests passed, including the SQLite context adapter test.
  • Rust e2e: not applicable.

@thekevinbot
thekevinbot enabled auto-merge October 6, 2026 11:43
…-cell

# Conflicts:
#	packages/rust/src/vtab.rs
#	testing-conventions.toml
@thekevinbot
thekevinbot merged commit 6a63a81 into main Oct 6, 2026
46 checks passed
@thekevinbot
thekevinbot deleted the claude/1330-vtab-stat-cell branch October 6, 2026 15:15
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