Skip to content

Fix Zookeeper bulk edit stale project handling#12229

Draft
harsh181018 wants to merge 2 commits into
mainfrom
harsh/fix-zk-systemio-stale-project
Draft

Fix Zookeeper bulk edit stale project handling#12229
harsh181018 wants to merge 2 commits into
mainfrom
harsh/fix-zk-systemio-stale-project

Conversation

@harsh181018

Copy link
Copy Markdown
Contributor

Summary

  • skip the bulk-delete project lookup when a Zookeeper/SystemIO bulk edit has no files to delete
  • add a SystemIO file-system failure callback so pending Zookeeper history writes can be cancelled
  • clear the Zookeeper history recording guard on bulk write failure so file watchers/re-exec do not stay suppressed

Testing

  • ./node_modules/.bin/vitest run --mode=development --project=integration src/machines/systemIO/systemIOMachine.spec.ts
  • ./node_modules/.bin/biome check --formatter-enabled=true --linter-enabled=false --organize-imports-enabled=true --files-ignore-unknown=true src/components/layout/areas/MlEphantConversationPaneWrapper.tsx src/machines/systemIO/systemIOMachine.spec.ts src/machines/systemIO/systemIOMachine.ts src/machines/systemIO/systemIOMachineImpl.ts

Notes

  • npm run tsc still fails in this clean worktree because generated Rust binding files under @rust/kcl-lib/bindings are missing; grep of the tsc log showed no errors in the touched files.

@vercel

vercel Bot commented Jun 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
modeling-app Ready Ready Preview, Comment Jun 25, 2026 8:09pm

Request Review

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