fix(ci): add eslint, lockfile and tests for central CI #3

Merged

forge-recovery merged 1 commits from ci/central-ci-repair-20261101 into develop 2026-11-01 19:56:11 +00:00

Conversation 0 Commits 1 Files Changed 9

+3225 -7

forge-recovery commented 2026-11-01 19:38:51 +00:00

Owner

Copy link

Copy source

Summary

Make the central CI profile pass for jarvis-file: it had no ESLint, no lockfile and no test files despite node --test scripts.

• Add ESLint 10 (recommended rules; Node CommonJS, browser globals for jarvis-file-core/js) and eslint.config.mjs.
• Add the first package-lock.json (newest versions within existing ranges; nothing was pinned before).
• Fix unused-binding lint errors using the repo's _ prefix convention; stub init calls are kept.
• Add 13 node:test tests (validate-path, rules-compiler, context-reader).

Verification

Pinned node:22.20.0-bookworm, base ff2104e: npm ci, lint, test pass (13 passed, 0 failed).
Note: ESLint 10 and the test glob need Node 22 (central CI uses 22); the old Node 18 matrix is not covered. Runtime source compatibility is unchanged.

🤖 Generated with Claude Code

## Summary Make the central CI profile pass for jarvis-file: it had no ESLint, no lockfile and no test files despite `node --test` scripts. - Add ESLint 10 (recommended rules; Node CommonJS, browser globals for `jarvis-file-core/js`) and `eslint.config.mjs`. - Add the first `package-lock.json` (newest versions within existing ranges; nothing was pinned before). - Fix unused-binding lint errors using the repo's `_` prefix convention; stub init calls are kept. - Add 13 `node:test` tests (validate-path, rules-compiler, context-reader). ## Verification Pinned `node:22.20.0-bookworm`, base `ff2104e`: `npm ci`, lint, test pass (13 passed, 0 failed). Note: ESLint 10 and the test glob need Node 22 (central CI uses 22); the old Node 18 matrix is not covered. Runtime source compatibility is unchanged. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

forge-recovery added 1 commit 2026-11-01 19:38:52 +00:00

fix(ci): add eslint, lockfile and tests for central CI ... 86646832e2

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

Co-Authored-By: Codex <noreply@openai.com>

forge-recovery merged commit ca2bc895fd into develop 2026-11-01 19:56:11 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

question

Further information is requested

wontfix

This will not be worked on

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

forge-recovery (Forge Recovery) james.maes (James Maes)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: KofTwentyTwo/jarvis-file#3