Skip to content

Codemod to convert component Args to Signature

License

Notifications You must be signed in to change notification settings

ijlee2/ember-codemod-args-to-signature

Repository files navigation

This project uses GitHub Actions for continuous integration.

ember-codemod-args-to-signature

Codemod to convert component Args to Signature1

1. Components without Args are also supported.

What is it?

To introduce Glint, you will need to write the signature and template registry for each component. This can be an error-prone, onerous task for large projects.

You can run this codemod to automate the required change.

Features

  • Scaffolds signature for components
  • Adds template registry for components
  • Preserves your code whenever possible
  • Focuses on maintainability and extensibility
A code diff to show what template-only components can look like before and after running ember-codemod-args-to-signature
Template-only components

A code diff to show what Glimmer components can look like before and after running ember-codemod-args-to-signature Another code diff for a Glimmer component
Glimmer components

Usage

Step 1. Quickly migrate.

cd <path/to/your/project>
npx ember-codemod-args-to-signature <arguments>

Step 2. Review the package.

  • Fill in missing type information.
  • Confirm that you can run all scripts in package.json.

For more information, please check the FAQ.

Prerequisites

Must:

  • Migrate to the Octane layout (flat or nested). You can leverage the codemods for classic and pod layouts.
  • If you have component classes written in JavaScript, change the file extension to *.ts to allow the codemod to add Signature and Registry.

Nice to do:

  • Refactor code (e.g. Glimmerize components, meet the linting rule no-implicit-this) to help the codemod parse code.

Arguments

You must pass --src to indicate the location of your components.

# Apps
npx ember-codemod-args-to-signature --src app/components

# V1 addons
npx ember-codemod-args-to-signature --src addon/components

# V2 addons
npx ember-codemod-args-to-signature --src src/components
Optional: Specify the component structure

By default, an Octane project has the flat component structure. Pass --component-structure to indicate otherwise.

npx ember-codemod-args-to-signature --component-structure nested
Optional: Specify the project root

Pass --root to run the codemod on a project somewhere else (i.e. not in the current directory).

npx ember-codemod-args-to-signature --root <path/to/your/project>

Limitations

The codemod is designed to cover typical cases. It is not designed to cover one-off cases. (Classic components are not supported.)

To better meet your needs, consider cloning the repo and running the codemod locally.

cd <path/to/cloned/repo>

# Compile TypeScript
pnpm build

# Run codemod
./dist/bin/ember-codemod-args-to-signature.js --root <path/to/your/project>

Frequently asked questions

How can I introduce Glint incrementally?

In a small project, we may go ahead with replacing tsc --noEmit, the default command for lint:types that is set by ember-cli, with the command glint.

/* package.json */
{
  "scripts": {
    "lint": "concurrently \"npm:lint:*(!fix)\" --names \"lint:\"",
    "lint:js": "eslint . --cache",
    "lint:js:fix": "eslint . --fix",
-     "lint:types": "tsc --noEmit"
+     "lint:types": "glint"
  }
}

However, in a large project (e.g. a monorepo with many packages), we will want to introduce Glint and fix incorrect types incrementally. A divide-and-conquer strategy will help you parallelize work, make smaller pull requests, and help others avoid merge conflicts.

So, instead, create a script called _lint:glint. This script helps you check if Glint is set up right and which errors exist already. The underscore also prevents the lint script from running Glint in continuous integration (since the command glint will likely fail at first).

/* package.json */
{
  "scripts": {
+     "_lint:glint": "glint",
    "lint": "concurrently \"npm:lint:*(!fix)\" --names \"lint:\"",
    "lint:js": "eslint . --cache",
    "lint:js:fix": "eslint . --fix",
    "lint:types": "tsc --noEmit"
  }
}

After you have fixed all type errors in the package, you can remove _lint:glint and use lint:types to run glint.

/* package.json */
{
  "scripts": {
-     "_lint:glint": "glint",
    "lint": "concurrently \"npm:lint:*(!fix)\" --names \"lint:\"",
    "lint:js": "eslint . --cache",
    "lint:js:fix": "eslint . --fix",
-     "lint:types": "tsc --noEmit"
+     "lint:types": "glint"
  }
}
Passing a signature to templateOnlyComponent() caused a type error with the message, Expected 0 type arguments, but got 1.. What should I do?

Likely, your project relies on ember-source@v3 and @types/ember__component@v3.

Unfortunately, @types/ember__component@v3 doesn't allow passing the signature to templateOnlyComponent() (we can see this from the file named template-only.d.ts), while @types/ember__component@v4 does. Updating the package to v4 isn't a viable option when you aren't on ember-source@v4.

Until you can update ember-source, I can provide three (3) fixes to temporarily address the type error. Each approach has pros and cons.

  1. Don't pass the signature to templateOnlyComponent(), but keep the signature in the file for reference.

    import templateOnlyComponent from '@ember/component/template-only';
    
    + // eslint-disable-next-line @typescript-eslint/no-unused-vars
    interface NavigationMenuSignature {
      // ...
    }
    
    - const NavigationMenuComponent = templateOnlyComponent<NavigationMenuSignature>();
    + const NavigationMenuComponent = templateOnlyComponent();
    
    export default NavigationMenuComponent;

    This will help you focus on updating import paths and fixing formatting issues, e.g. by running the scripts lint:types, lint:js:fix, and lint:js.

    The downsides? Assuming the @glint-ignore directive is not used, the glint command (the _lint:glint script) can never pass, since template-only components don't provide a signature. Because the signature is missing, ember-codemod-args-to-signature will create an extra Signature if you run the codemod again.

  2. Create an empty backing class.

    - import templateOnlyComponent from '@ember/component/template-only';
    + import Component from '@glimmer/component';
    
    interface NavigationMenuSignature {
      // ...
    }
    
    - const NavigationMenuComponent = templateOnlyComponent<NavigationMenuSignature>();
    
    - export default NavigationMenuComponent;
    + export default class NavigationMenuComponent extends Component<NavigationMenuSignature> {}

    The glint command can now pass potentially. However, the components are no longer template-only. This might be a concern if maintaining a high performance is a factor.

  3. Patch @types/ember__component@v3.

    If you have experience with patching dependencies, you can patch the file template-only.d.ts so that its code matches that from @types/ember__component@v4. Note, there may be multiple files of template-only.d.ts that you will need to patch.

    The glint command can pass potentially and the components remain template-only. However, it's unclear if there are files in ember-source@v3 that assume template-only.d.ts to provide types in the way that it does in v3.

Compatibility

  • Node.js v18 or above

Contributing

See the Contributing guide for details.

If you have an open-sourced project that I can use for testing, reach out to me on Discord at ijlee2. Please star this project so that I can gauge its importance to you and the Ember community. ⭐

License

This project is licensed under the MIT License.