Replace ESLint with Biome

[gaurav]

Used Claude to replace ESLint (with Airbnb base style) with Biome because the Airbnb base style does not appear to be maintained. This will bring this repo in line with phyx.js (phyloref/phyx.js#156).

Also removed the push trigger in GitHub Actions -- the pull_request trigger covers all of our use cases.

[Copilot]

Pull request overview

This PR replaces ESLint (Airbnb base + plugins) with Biome as the repository linter, updates linting configuration, and applies code edits to satisfy the new lint rules across tests and scripts.

Changes:

• Swap eslint-based linting for biome lint and add biome.json configuration.
• Update various JS files (tests and scripts) to match Biome’s recommended rules (e.g., node: core imports, arrow callbacks / for...of).
• Adjust CI Node matrix and workflow triggers.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
package.json	Replaces eslint lint script/deps with Biome.
package-lock.json	Updates lockfile to remove ESLint deps and add Biome + platform CLIs.
biome.json	Adds Biome configuration (recommended lint rules, formatter disabled).
.eslintrc.json	Removes root ESLint configuration.
test/.eslintrc.json	Removes test-only ESLint overrides.
test/test_phyx2ontology.js	Updates Node core imports and Mocha callbacks for linting compatibility.
test/test_phyx.js	Refactors iteration/callback style for linting compatibility.
test/regnum2phyx/exec.js	Refactors iteration/callback style for linting compatibility.
regnum2phyx/regnum2phyx.js	Refactors loops/imports; adjusts citation field handling.
phyx2ontology/phyx2ontology.js	Refactors loops/imports and replaces inner-callback returns with continue.
.github/workflows/test.yaml	Updates Node matrix and modifies workflow triggers.

Comments suppressed due to low confidence (1)

test/test_phyx.js:104

• assert.true(result) is not part of Chai’s assert API (the usual assertion is assert.isTrue(result) / assert.strictEqual(result, true)). This is currently inside a skipped test, but it will fail immediately if the .skip is removed as the comment suggests.

        const result = phyxSchema(json);
        const errorStrings = (phyxSchema.errors || []).map(err => ajvInstance.errorsText([err]));
        assert.deepEqual(errorStrings, []);
        assert.isNull(phyxSchema.errors);
        assert.true(result);
      });

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[hlapp]

Haven't taken a deep look, but apparently Copilot did, and those comments got addressed. I'm not seeing anything else jump out at me, so 👍🏻