Skip to content

build: add eslint config and lint script - #12

Merged
TheGreatAxios merged 2 commits into
cl-9107-embedding-replace-hand-rolled-fetchscheduler-fakes-withfrom
cl-9069-embedding-add-eslint-config-and-lint-script
Sep 27, 2026
Merged

TheGreatAxios merged 2 commits into
cl-9107-embedding-replace-hand-rolled-fetchscheduler-fakes-withfrom
cl-9069-embedding-add-eslint-config-and-lint-script

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Verification

  • bun run lint, prettier, typecheck, build and bun run test (16 pass); CI passes.

Closes CL-9069

@TheGreatAxios TheGreatAxios left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review

Independent review of the stack found: bad base64 replies escaped as a raw DOMException or silently truncated vectors; non-integer indices were accepted; duplicate-index and non-JSON-200 coverage was lost; the live suite failed instead of skipping when the model was absent. All fixed in this stack (strict base64 decode, number.integer index, restored harness tests, model-aware skip).

Remaining notes: peer range >=0.4.0 follows sibling convention. Each commit typechecks, tests, and builds standalone.

@TheGreatAxios
TheGreatAxios force-pushed the cl-9069-embedding-add-eslint-config-and-lint-script branch from ff0c099 to 6103d63 Compare September 25, 2026 13:50
@TheGreatAxios
TheGreatAxios force-pushed the cl-9069-embedding-add-eslint-config-and-lint-script branch from 6103d63 to e08181c Compare September 25, 2026 13:58
@TheGreatAxios
TheGreatAxios force-pushed the cl-9069-embedding-add-eslint-config-and-lint-script branch from e08181c to e0a1d6e Compare September 25, 2026 14:08
@TheGreatAxios
TheGreatAxios force-pushed the cl-9069-embedding-add-eslint-config-and-lint-script branch from e0a1d6e to 6ae2c4d Compare September 25, 2026 15:14
@TheGreatAxios
TheGreatAxios force-pushed the cl-9069-embedding-add-eslint-config-and-lint-script branch from 6ae2c4d to 301ff4b Compare September 25, 2026 17:05
@TheGreatAxios
TheGreatAxios force-pushed the cl-9069-embedding-add-eslint-config-and-lint-script branch from 301ff4b to ae0c32e Compare September 25, 2026 17:36
@TheGreatAxios
TheGreatAxios force-pushed the cl-9069-embedding-add-eslint-config-and-lint-script branch from ae0c32e to 9c43c26 Compare September 25, 2026 23:48
@TheGreatAxios
TheGreatAxios force-pushed the cl-9069-embedding-add-eslint-config-and-lint-script branch from 9c43c26 to 81edd7e Compare September 25, 2026 23:49
@TheGreatAxios TheGreatAxios changed the title eslint config and lint script Add eslint config and lint script Sep 25, 2026
Matches corbits-ollama-adapter's config and tool versions. CI runs lint
and checks prettier across the whole repo; the markdown is formatted.
@TheGreatAxios
TheGreatAxios force-pushed the cl-9069-embedding-add-eslint-config-and-lint-script branch from 81edd7e to 7a16314 Compare September 26, 2026 01:29
@TheGreatAxios TheGreatAxios changed the title Add eslint config and lint script build: add eslint config and lint script Sep 26, 2026
@TheGreatAxios
TheGreatAxios added this pull request to stack #15 September 26, 2026 01:58
@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Superseded: oxfmt and oxlint replace ESLint and Prettier in this repo.

@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Reopened only so the stack merges in order: later PRs are built on this branch. The oxfmt/oxlint PR higher in the stack removes this ESLint setup, so main ends up with oxc only.

@TheGreatAxios
TheGreatAxios merged commit 0222794 into main Sep 27, 2026
2 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.

1 participant