Commit 8bd83b7

committed

refactor(db): replace flow_variable prev/next linked list with display_order

Replace the prev/next linked list implementation for flow variable ordering with a simple display_order float field. This simplifies the codebase and aligns with the TypeSpec API contract. Changes: - Update flow_variable schema to use display_order instead of prev/next - Simplify SQL queries (remove recursive CTE, use ORDER BY display_order) - Remove FlowVariableMovableRepository and linked list manipulation logic - Update model, service, and RPC layers to use Order field directly - Frontend now provides order value directly (no server-side generation)

1 parent 6c49ae2 commit 8bd83b7Copy full SHA for 8bd83b7

14 files changed

packages
- db
  - pkg/sqlc
    - gen
    - queries
      - flow.sql
    - schema
      - 05_flow.sql
    - sqlc.yaml
  - test_variable_ordering.sql
- server
  - internal/api/rflowv2
  - pkg
    - model/mflowvariable
      - mflowvariable.go
    - service/sflowvariable
      - repository.go
      - sflowvariable.go

`‎packages/db/pkg/sqlc/gen/db.go‎`

Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Comments

(0)