header-gen: Remove node-ts & update cli script. #9
Conversation
| import { headerData } from "./inputdata.js"; | ||
| import { generateBlob, generateCArray, generateDsHexLiteral } from "./main.js"; |
There was a problem hiding this comment.
@microbit-robert this is the part I haven't had a chance to look into.
As npm run start ends up running node built/cli.js it looks like the script needs the import to include the extension.
Not sure how to get ts to convert the import statements (in which case maybe we need to reintroduce tsconfig.cli.json and npm start would then be tsc --project tsconfig.cli.json && node built/cli.js.
There was a problem hiding this comment.
Assuming the ml extension needs the import statements in the built output as they currently are, then we'd need to make sure the packaging script still used the original tsconfig.json and excludes the cli.ts file from the package (as that'd be broken to be used with node and it doesn't do anything as an imported module).
As I believe package.json "type": "module" forces ES Modules syntax.