Skip to content
This repository was archived by the owner on Jan 6, 2026. It is now read-only.

Improve and consolidate package.json scripts#1757

Merged
wycats merged 1 commit intomainfrom
cleanup-unused-scripts
Jul 1, 2025
Merged

Improve and consolidate package.json scripts#1757
wycats merged 1 commit intomainfrom
cleanup-unused-scripts

Conversation

@wycats
Copy link
Copy Markdown
Contributor

@wycats wycats commented Jul 1, 2025

Summary

This PR builds on #1756 to further improve the package.json scripts by consolidating duplicates and standardizing naming.

Changes

  • Created bin/ts wrapper script for TypeScript execution with --experimental-strip-types
  • Consolidated lint scripts:
    • Removed repo:lint:fix (duplicate of lint:fix)
    • Removed repo:lint:files (redundant)
    • Renamed test:lintlint for consistency
    • Standardized all linting under lint:* prefix
  • Removed unused scripts and files:
    • link:all and unlink:all scripts
    • bin/link-all.mts and bin/unlink-all.mts files
    • esyes dependency (only used by removed scripts)
  • Updated all TypeScript execution scripts to use ./bin/ts wrapper
  • Updated turbo.json to match renamed lint task
  • Updated CLAUDE.md documentation with new script names

Benefits

  • Cleaner, more consistent script naming
  • No more repeated --experimental-strip-types flags
  • Better separation between lint and test commands
  • Reduced dependencies

Test plan

  • Verified all scripts work correctly
  • Ran CI preparation commands successfully
  • Updated all references in turbo.json and documentation

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 1, 2025

This PRmain
Dev
588K └─┬ .
169K   ├── runtime
160K   ├── syntax
100K   ├── compiler
 58K   ├── opcode-compiler
 27K   ├── manager
 24K   ├── validator
 11K   ├── program
8.9K   ├── reference
7.2K   ├── destroyable
6.3K   ├── util
4.3K   ├── node
3.4K   ├── global-context
2.5K   ├── wire-format
1.0K   ├── vm
969B   ├── encoder
844B   ├── vm-babel-plugins
606B   └── owner
588K └─┬ .
169K   ├── runtime
160K   ├── syntax
100K   ├── compiler
 58K   ├── opcode-compiler
 27K   ├── manager
 24K   ├── validator
 11K   ├── program
8.9K   ├── reference
7.2K   ├── destroyable
6.3K   ├── util
4.3K   ├── node
3.4K   ├── global-context
2.5K   ├── wire-format
1.0K   ├── vm
969B   ├── encoder
844B   ├── vm-babel-plugins
606B   └── owner
Prod
231K └─┬ .
 70K   ├── syntax
 63K   ├── runtime
 48K   ├── compiler
 18K   ├── opcode-compiler
7.9K   ├── manager
5.1K   ├── validator
4.8K   ├── program
3.6K   ├── reference
2.4K   ├── util
2.1K   ├── node
1.6K   ├── wire-format
1.5K   ├── destroyable
737B   ├── vm
594B   ├── global-context
516B   ├── encoder
469B   ├── vm-babel-plugins
155B   └── owner
231K └─┬ .
 70K   ├── syntax
 63K   ├── runtime
 48K   ├── compiler
 18K   ├── opcode-compiler
7.9K   ├── manager
5.1K   ├── validator
4.8K   ├── program
3.6K   ├── reference
2.4K   ├── util
2.1K   ├── node
1.6K   ├── wire-format
1.5K   ├── destroyable
737B   ├── vm
594B   ├── global-context
516B   ├── encoder
469B   ├── vm-babel-plugins
155B   └── owner

Comment thread bin/link-all.mts
- Create bin/ts wrapper for TypeScript execution with --experimental-strip-types
- Consolidate lint scripts:
  - Remove duplicate repo:lint:fix (same as lint:fix)
  - Remove redundant repo:lint:files
  - Rename test:lint → lint for consistency
  - Standardize all linting under lint:* prefix
- Keep link:all and unlink:all scripts for testing against ember.js
  - Add documentation explaining their purpose
  - Update to use bin/ts wrapper
- Update all TypeScript execution scripts to use ./bin/ts wrapper
- Update turbo.json and CI workflows to match renamed tasks
- Update CLAUDE.md and eslint.config.js with necessary changes
- Remove esyes dependency (only used by link scripts, now using bin/ts)

Benefits:
- Cleaner, more consistent script naming
- No more repeated --experimental-strip-types flags
- Better separation between lint and test commands
- Reduced dependencies
@wycats wycats force-pushed the cleanup-unused-scripts branch from 94a386e to 951f928 Compare July 1, 2025 04:47
@wycats wycats merged commit 268fdea into main Jul 1, 2025
9 checks passed
@wycats wycats deleted the cleanup-unused-scripts branch July 1, 2025 04:55
@github-actions github-actions Bot mentioned this pull request Jul 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants