Fix node button reactivity by using Praxis store dispatch

[Copilot]

• Identify the root cause of node buttons not working
• Fix reactivity in canvas-praxis.ts by using spread operator for array updates
• Use praxisStore.dispatch() to properly notify subscribers
• Ensure single store instance is used across modules
• Remove debug logging
• Fix missing canvasEngine import
• Test that Terminal node button works
• Test other node buttons (Input, Display, Transform)
• Run existing tests to ensure no regressions (81 tests pass)
• Run code review (no issues found)
• Run security scan (no vulnerabilities found)
• Address code review feedback: rename praxisStoreInternal to canvasPraxisStoreInstance
• Address code review feedback: remove unused imports from canvas.ts

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Add node buttons didn't work</issue_title>
<issue_description>None of the node type buttons worked. </issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Add node buttons didn't work #24

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull request overview

Fixes non-reactive node toolbar buttons by ensuring canvas updates go through the Praxis Svelte store so subscribers are notified consistently.

Changes:

• Route all canvas actions through praxisStoreInternal.dispatch() instead of calling canvasEngine.step() directly.
• Update add-node / add-connection rules to use immutable array reassignment to support reactivity.
• Reuse a single Praxis store instance from canvas-praxis.ts in canvas.ts to avoid duplicate store instances.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/lib/stores/canvas.ts	Reuses the shared Praxis store instance instead of creating a second store.
src/lib/stores/canvas-praxis.ts	Adds exported store instance and switches actions/rules to patterns that trigger subscriber updates.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

CanvasContext and CanvasNode are imported but no longer used in this module after switching to praxisStoreInternal. If svelte-check/TS noUnusedLocals is enabled (common in SvelteKit), this will fail the check step. Remove the unused imports or use them for explicit typing where needed.

Suggested change

	import { canvasPraxisStore, praxisStoreInternal, canvasEngine, type CanvasContext } from './canvas-praxis';
	import type { Canvas, CanvasNode, Connection } from '../types/canvas';
	import { canvasPraxisStore, praxisStoreInternal, canvasEngine } from './canvas-praxis';
	import type { Canvas, Connection } from '../types/canvas';

[Copilot]

Removed unused CanvasContext and CanvasNode type imports in commit 6fceead.

[kayodebristol]

@copilot apply changes based on the comments in this thread

[kayodebristol]

@copilot apply changes based on the comments in this thread

[Copilot]

@copilot apply changes based on the comments in this thread

All changes from the code review have already been applied in commit 6fceead:

• Renamed praxisStoreInternal to canvasPraxisStoreInstance
• Removed unused CanvasContext and CanvasNode type imports from canvas.ts

All tests pass (81 tests) and there are no TypeScript errors.