Skip to content

feat: integrate component and board variables - #19

Merged
panayotisk merged 51 commits into
mainfrom
AR-3186-update-component-board-variable-integration
Sep 2, 2026
Merged

panayotisk merged 51 commits into
mainfrom
AR-3186-update-component-board-variable-integration

Conversation

@Zanzavar

@Zanzavar Zanzavar commented Aug 2, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • make TypeScript unqualified lookups consistently resolve global variables while preserving qualified board/component scopes
  • generalize the C# project model and runtime enumeration to include global, board, and component variables, including ID-based reset exclusions
  • split C++ current and default values in the exported input contract so reset and resetAll restore project defaults across interpreter invocations
  • reject null or missing runtime values with explicit validation errors and add regression coverage across all three implementations
  • make CI deterministic with Node 22, pnpm 10.12.2, Java 17, and a Windows 2022/Visual Studio 2022 native build

Related work

  • Task: AR-3186
  • Depends on arcweave/arcweave#1665
  • Supports the board-variable schema migration in arcweave/arcweave#1701

Testing

  • ./node_modules/.bin/tsc --build
  • ./node_modules/.bin/vitest --run — 158 tests passed
  • ./node_modules/.bin/eslint src
  • dotnet test CSharp.sln --configuration Release --verbosity normal — 145 tests passed, 0 warnings
  • GitHub Actions TypeScript build and package preview — passed
  • GitHub Actions C# build and tests — passed
  • GitHub Actions C++ build and tests on macOS and Windows 2022 — passed

Migration and configuration impact

  • Breaking ABI change: C++ consumers must update the exported UVariable header, marshal both current and default values, and rebuild bundled native libraries
  • C# consumers must provide component collections and can enumerate all variables through GetAllVariables()
  • CI now pins Node 22, pnpm 10.12.2, Java 17, and Windows 2022 for reproducible builds
  • No application configuration or database migration changes

Screenshots

Not applicable; this change has no UI.

@pkg-pr-new

pkg-pr-new Bot commented Aug 2, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@arcweave/arcscript@19

commit: 3b9ccd8

@Zanzavar
Zanzavar marked this pull request as ready for review August 31, 2026 17:23
@panayotisk
panayotisk merged commit d94a767 into main Sep 2, 2026
6 checks passed
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