Skip to content

Conversation

@deadronos
Copy link
Owner

Align instruction files to reflect current best practices, standardizing on pnpm and removing outdated content. Update package dependencies and refactor the refinery system for improved logic separation. Document changes in ALIGN-INSTRUCTIONS.md.

google-labs-jules bot and others added 9 commits January 3, 2026 18:10
- Standardized on `pnpm` across all docs and scripts.
- Consolidated and cleaned up `spec-driven-workflow-v1.instructions.md`.
- Updated `nodejs-javascript-vitest` and `typescript-5-es2022` to reflect current toolchain.
- Aligned `playwright-typescript` with actual configuration.
- Removed legacy `powershell` and `tasksync` instructions.
- Fixed a lint error in `vitest.config.ts`.
- Created `ALIGN-INSTRUCTIONS.md` to document the changes.
- Standardized on `pnpm` across all docs and scripts.
- Consolidated `spec-driven-workflow-v1.instructions.md`.
- Updated `nodejs-javascript-vitest` and `typescript-5-es2022` to reflect current toolchain.
- Aligned `playwright-typescript` with actual configuration.
- Removed legacy `powershell` and `tasksync` instructions.
- Fixed ESLint warnings in `droneAI.ts`, `rustSnapshot.ts`, `store.ts`, `droneSlice.ts`.
- Replaced legacy `eslint-env` with `global` in generator template.
- Fixed `vitest.config.ts` type error.
- Verified with `pnpm lint`, `pnpm typecheck`, `pnpm test`.
- Add `applyRefineryStats` action to store to handle resource updates
- Update `createRefinerySystem` ECS system to compute stats locally using `computeRefineryProduction` and then commit via `applyRefineryStats`
- Remove dependency on `processRefinery` store action within the ECS loop (though it remains for offline processing)
- Bump @typescript-eslint/eslint-plugin and @typescript-eslint/parser to ^8.52.0
- Update globals to ^17.0.0
- Upgrade rollup to ^4.55.1
- Update sass-embedded to ^1.97.2
- Bump wasm-pack to ^0.13.1
…279243515991948

Refactor Refinery System Logic
@deadronos deadronos merged commit d897b2a into dev Jan 7, 2026
3 of 4 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