Skip to content

Rename npm build script to compile #8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
AndreasArvidsson merged 2 commits into from
Aug 1, 2026
Merged

Rename npm build script to compile #8

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
feeb713
Re named "build" script to "compile"
AndreasArvidsson Aug 1, 2026
6119d92
Update readme
AndreasArvidsson Aug 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/deploy.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,8 @@ jobs:
- name: Install dependencies
run: npm --color ci

- name: Build
run: npm --color run build
- name: Build dist
run: npm --color run compile

- name: Read version from package.json
run: |
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/test.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ jobs:
- name: Install dependencies
run: npm --color ci

- name: Build
run: npm --color run build
- name: Build dist
run: npm --color run compile

- name: Run lint
run: npm --color run lint
Expand Down
4 changes: 2 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ foo bar baz: "foo bar baz"
```yaml
repos:
- repo: https://github.com/cursorless-dev/talon-tools
rev: v0.11.0
rev: v0.11.2
hooks:
- id: talon-fmt
- id: snippet-fmt
Expand Down Expand Up @@ -147,4 +147,4 @@ When creating a new release do the following:

- Update version in [`package.json`](./package.json)
- Update version in [`README.md`](./README.md#pre-commit)
- Run build: `npm run build`
- Run build/compile: `npm run compile`
2 changes: 1 addition & 1 deletion dist/snippetFormatter.js
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions dist/snippetFormatter.js.map
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/talonFormatter.js
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions dist/talonFormatter.js.map
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/treeSitterFormatter.js
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions dist/treeSitterFormatter.js.map
View file
Open in desktop

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 4 additions & 5 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,14 +1,13 @@
{
"name": "@cursorless/talon-tools",
"version": "0.11.1",
"version": "0.11.2",
"description": "Linting and formatting tools for Talon and Cursorless",
"author": "Cursorless Dev",
"license": "MIT",
"type": "module",
"files": [
"dist/**/*.js",
"dist/**/*.d.ts",
"!dist/test/**",
"dist",

@chatgpt-codex-connector chatgpt-codex-connector Bot Aug 1, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restore the specific dist file allowlist

When .github/workflows/deploy.yaml publishes this package, the broader dist entry includes all five generated .js.map files, reversing the immediately preceding “More specific files list” change. I verified with npm pack --dry-run --json that this increases the packed artifact from 17,712 to 75,679 bytes and the unpacked size from 68,977 to 300,311 bytes. Keep the dist/**/*.js and dist/**/*.d.ts allowlist so every release does not ship these unintended source maps.

Useful? React with 👍 / 👎.

"!dist/test",
"!dist/build.*"
],
"types": "./dist/lib.d.ts",
Expand Down Expand Up @@ -36,7 +35,7 @@
"url": "https://github.com/sponsors/cursorless-dev"
},
"scripts": {
"build": "npm run clean && tsc -p . && tsx ./src/build.ts",
"compile": "npm run clean && tsc -p . && tsx ./src/build.ts",
"clean": "rm -rf dist/*",
"lint": "npm run typecheck && npm run lint:ts && npm run lint:fmt",
"typecheck": "tsc -p . --noEmit",
Expand Down