Skip to content

Conversation

@initstring
Copy link
Owner

🔧 Dependency Maintenance & React Flow Migration

Summary
This PR performs essential dependency maintenance, primarily migrating the deprecated reactflow library to its successor @xyflow/react and fixing testing configurations to align with updated tooling.

Key Changes

1. React Flow Migration

  • Replaced reactflow (v11) with @xyflow/react (v12) throughout the application.
  • Updated imports in technique-node.tsx and attack-flow/index.tsx.
  • Updated CSS import to @xyflow/react/dist/style.css.
  • Type Fixes: Resolved strict TypeScript errors exposed by the new library version (e.g., ensuring Node data satisfies Record<string, unknown>).

2. Test Configuration

  • Fixed vitest.config.ts breakage caused by Vitest upgrades. replaced the deprecated poolOptions with fileParallelism: false to ensure stable, single-threaded database testing.

3. General Updates

  • Verified zod compatibility (v4 check).
  • confirmed npm audit is clean after updates.

Testing

  • Verified npm run check (TypeScript/ESLint) passes.
  • Verified npm run test passes (49/49 test suites).
  • Manual check: Attack Flow visualization loads correctly with new library.

- Replace deprecated reactflow with @xyflow/react
- Fix Vitest config for v2+ compatibility
- Update import paths and fix TypeScript type errors
@initstring initstring self-assigned this Dec 25, 2025
@initstring
Copy link
Owner Author

@codex please review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@initstring initstring merged commit b83498f into main Dec 25, 2025
5 checks passed
@initstring initstring deleted the apply-patches-dec-2025 branch December 25, 2025 22:34
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.

2 participants