Petrinaut: Remove linter warnings

[kube]

🌟 What is the purpose of this PR?

Clean up Petrinaut lint diagnostics after the React Compiler/Oxlint updates, keeping behavior equivalent while replacing duplicated or lint-hostile patterns with clearer local helpers.

🔍 What does this change?

• Adds small React helpers for latest values, stable callbacks, source-keyed draft fields, and subscription-backed value stores.
• Refactors editor panels/components to avoid stale prop-to-state sync, unsafe ref access, and repeated validation input code.
• Makes Spreadsheet controlled and clamps selection/focus/editing state against current data.
• Tightens story-provider, drawer, resize, LSP, and timeline effects so subscriptions and cleanup are explicit.
• Promotes Petrinaut React Compiler plugin diagnostics from warnings to errors.
• Documents intentional lint/compiler opt-outs where the code relies on them.

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

• does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

• are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

• do not affect the execution graph

🛡 What tests cover this?

• yarn workspace @hashintel/petrinaut lint:tsc
• yarn workspace @hashintel/petrinaut lint:eslint
• yarn workspace @hashintel/petrinaut test:unit

❓ How to test this?

1. Run the checks above.
2. Spot-check Petrinaut editor flows touched by the cleanup: property-panel field edits, spreadsheet initial-state edits, left-sidebar/search keyboard navigation, and simulation timeline playback.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
hash	Ready	Preview, Comment	May 20, 2026 8:45am
petrinaut	Ready	Preview, Comment	May 20, 2026 8:45am

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
hashdotdesign-tokens	Ignored	Preview	May 20, 2026 8:45am

[cursor]

PR Summary

Medium Risk
Moderate risk due to refactors in interactive editor UI (controlled Spreadsheet, selection/focus clamping, and new subscription-based state stores) that could subtly change runtime behavior despite aiming for equivalence.

Overview
Promotes React Compiler-related react-hooks-js/* oxlint rules to errors and refactors Petrinaut to comply, replacing render-time ref writes/prop→state syncing patterns with safer primitives.

Adds small helpers (createValueStore + useSyncExternalStore integration, useDraftField, DraftFieldInput, and clampIndex) and applies them across the editor: property panel name/identifier fields now share draft+validation behavior, list/search keyboard focus uses index clamping instead of syncing effects, and Spreadsheet becomes fully controlled (parent-owned data) with local UI state clamped against current rows.

Tightens lifecycle/cleanup in a few places (e.g. LSP client provider, drawer mounting, simulation timeline streaming store, Storybook provider subscriptions, resize drag callback refs) and documents/isolates intentional compiler/lint opt-outs (e.g. runtime visualizer component creation, useFiringDelta ref read).

^{Reviewed by Cursor Bugbot for commit cc9da36. Bugbot is set up for automated code reviews on this repo. Configure here.}

[augmentcode]

This pull request is abnormally large and would use a significant amount of tokens to review. If you still wish to review it, comment "augment review" and we will review it.

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

There are 3 total unresolved issues (including 1 from previous review).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4aca92d. Configure here.}

See the output of git range-diff at https://github.com/hashintel/hash/actions/runs/26151312720