content-tag's process incorrectly converts type imports to value imports.

[NullVoxPopuli]

This comes out of: embroider-build/addon-blueprint#195

Repro:

import { Preprocessor } from 'content-tag';
const p = new Preprocessor();


const gjs2 = `
import Component from '@glimmer/component';
import type { WithBoundArgs } from '@glint/template';
import { hash } from '@ember/helper';

import Header from './header.gts';
import Content from './content.gts';

export interface AccordionItemSignature {
  Element: HTMLDivElement;
  Blocks: {
    default: [
      {
        Header: WithBoundArgs<
          typeof Header,
          'value' | 'isExpanded' | 'toggle' | 'disabled'
        >;
        Content: WithBoundArgs<typeof Content, 'value' | 'isExpanded'>;
      },
    ];
  };
}

export default class AccordionItem extends Component<AccordionItemSignature> {
  /* ... */
  <template>
    ...
  </template>
}
`;

console.log(p.process(gjs2));

results in:

import { template } from "@ember/template-compiler";
import Component from '@glimmer/component';
import { WithBoundArgs } from '@glint/template';
import { hash } from '@ember/helper';
import Header from './header.gts';
import Content from './content.gts';
export interface AccordionItemSignature {
    Element: HTMLDivElement;
    Blocks: {
        default: [{
                Header: WithBoundArgs<typeof Header, 'value' | 'isExpanded' | 'toggle' | 'disabled'>;
                Content: WithBoundArgs<typeof Content, 'value' | 'isExpanded'>;
            }];
    };
}
export default class AccordionItem extends Component<AccordionItemSignature> {
    /* ... */ static{
        template(`
    ...
  `, {
            component: this,
            eval () {
                return eval(arguments[0]);
            }
        });
    }
}

not that the type import on @glint/template was stripped.

during the babel phase of the v2-addon build, this causes just WithBoundArgs to be removed, leaving a side-effecting import '@glint/template' -- because it's not safe to remove the entirety of value imports (which is why we want to require type annotations on type-imports.

[ef4]

I would guess this is a bug in the swc printer.

[NullVoxPopuli]

During full compilation, they at least handle it correctly

https://play.swc.rs/?version=1.3.74&code=H4sIAAAAAAAAA3VSy2rDMBC8%2Byv2ZgjBvschNGkDKbSnFnooPSjW2hbVw0hySEn9793Iip2UVhdpNLO7o0FCtcZ6uDe0a9QeKmsUpHe1FEqhzcsLkRaJGLT%2Bq0U4wZvwzcZ0mq9t7aC%2FKtQ%2B96hayTxOVSdomGsmHao9tW9QtmhJdZHtkHG0UZQRf4ZZ7d3U6d5oPznNyGLAUZTgMajIBdqKlQjrsjSWC6MfydWLqDXznaUXJABbiYpKF7B7fX56EIeIC6I20pSfbhFkABwr1kkSvgcI8fq8BseL2zyWIw0hL1NF3fyKSA9MdpjCN6TCbY8t0xx5gN7UtRwYLhzbS7ofC1fFeIxR%2FB4eJ0Z2%2Fs%2BgsU8%2FmPo4475I%2BjHD%2BGooJXPuNkfAI%2FXmbvo5y79zXoWo8hlkWQazPOl%2FAMKibKhyAgAA&config=H4sIAAAAAAAAA2WPwQqDQAxE736F5GxBLPTQf%2Bi190WzZYtuJIlQEf%2FdrFax9JZM5s2QKctzeEsN93yy0RZlF8UTd4dkYoM1sVPiJ7IEinaDqqyqS3mF1TMXG907FuQzKmNU90mAjj1KzaFXKP6CxRzKA%2F6kqeMXamJRUt2Xg5ZI0GTvWsGkrQB01Awtnl6xwg2%2BwWEK8thtqW8DQwx%2B3AOzeQEzww6KFgEAAA%3D%3D

Tho, I could still believe this information could accidentally be lost during printing:

[ef4]

swc-project/swc#8070