Skip to content

chore: migrate to pnpm and oxlint/oxfmt - #14

Merged
mokevnin merged 3 commits into
mainfrom
chore/standards
Aug 12, 2026
Merged

chore: migrate to pnpm and oxlint/oxfmt#14
mokevnin merged 3 commits into
mainfrom
chore/standards

Conversation

@mokevnin

Copy link
Copy Markdown
Contributor

Brings the repository in line with the Hexlet-wide toolchain. This was the last repository still on ESLint; everything else in the organisation is on oxlint and oxfmt.

  • pnpm replaces npm, oxlint + oxfmt replace ESLint.
  • frontend becomes a workspace package. It has its own package.json and was installed by a postinstall running cd frontend && npm ci; under pnpm that aborts, since the sub-project has no lockfile. One install at the root now covers both.
  • The package is marked private. It claimed the npm name @hexlet/code, which js-react-hexlet-chat also claimed. Neither is published, and two repositories pointing at one name is a trap for whoever wires up publishing next.

make build produces the production bundle, both packages lint clean.

mokevnin and others added 3 commits August 11, 2026 21:41
This was the last repository still on ESLint; everything else in the
organisation moved to oxlint and oxfmt.

frontend has its own package.json and was installed by a postinstall running
`cd frontend && npm ci`. Under pnpm that fails: the sub-project has no
pnpm-lock.yaml, so --frozen-lockfile aborts. It is declared a workspace package
instead, so one install at the root covers both and the postinstall goes away.
The build script loses npm's --prefix, which pnpm does not have, for -C.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Node CI keeps running on main as well as on pull requests: this repository
publishes nothing, so there is no release workflow to check main instead.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The manifest claimed the name @hexlet/code, which js-react-hexlet-chat also
claimed. Neither is published, and leaving two repositories pointing at one npm
name is a trap for whoever wires up publishing next.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@mokevnin
mokevnin merged commit c5c036b into main Aug 12, 2026
1 check passed
@mokevnin
mokevnin deleted the chore/standards branch August 12, 2026 01:42
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