build: add eslint config and lint script - #12
Conversation
TheGreatAxios
left a comment
There was a problem hiding this comment.
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.
ff0c099 to
6103d63
Compare
6103d63 to
e08181c
Compare
e08181c to
e0a1d6e
Compare
e0a1d6e to
6ae2c4d
Compare
6ae2c4d to
301ff4b
Compare
301ff4b to
ae0c32e
Compare
ae0c32e to
9c43c26
Compare
9c43c26 to
81edd7e
Compare
Matches corbits-ollama-adapter's config and tool versions. CI runs lint and checks prettier across the whole repo; the markdown is formatted.
81edd7e to
7a16314
Compare
|
Superseded: oxfmt and oxlint replace ESLint and Prettier in this repo. |
|
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. |
Summary
eslint.config.tsmatches corbits-ollama-adapter; eslint, @eslint/js, typescript-eslint and jiti at the same versions;lintscript.bun run lintandprettier --check .; markdown is formatted and IMPLEMENTATION.md lists the eslint dev dependencies.Verification
bun run lint, prettier, typecheck, build andbun run test(16 pass); CI passes.Closes CL-9069