Inject CLI version to dist after building

Author: eddeee888

.github/workflows/main.yml

@@ -79,8 +79,6 @@ jobs:
         run: |
           pnpm run dev-test:generate:${{matrix.method}}
           git diff --exit-code -- dev-test/
-      - name: Test CLI version ${{matrix.method}}
-        run: pnpm run dev-test:test-cli-version:${{matrix.method}}
 
   examples-tests:
     name: Examples - Normal

.gitignore

@@ -21,6 +21,3 @@ website/public/_redirects
 
 **/cypress/screenshots
 **/cypress/videos
-
-# Generated file for `graphql-codegen --version` to work. See `pnpm prepare-cli-version`.
-packages/graphql-codegen-cli/src/_version.ts

dev-test/general/package.json

@@ -7,8 +7,6 @@
     "generate": "pnpm generate:esm",
     "generate:cjs": "node ../../packages/graphql-codegen-cli/dist/cjs/bin.js --require dotenv/config --config codegen.ts dotenv_config_path=.env",
     "generate:esm": "node ../../packages/graphql-codegen-cli/dist/esm/bin.js --require dotenv/config --config codegen.ts dotenv_config_path=.env",
-    "test-cli-version:cjs": "node ../../packages/graphql-codegen-cli/dist/cjs/bin.js --version",
-    "test-cli-version:esm": "node ../../packages/graphql-codegen-cli/dist/esm/bin.js --version",
     "watch": "pnpm watch:esm",
     "watch:cjs": "pnpm generate:cjs --watch",
     "watch:esm": "pnpm generate:esm --watch"

eslint.config.mjs

@@ -83,6 +83,7 @@ export default [
       'dev-test/**',
       'website/**',
       'examples/**',
+      'scripts/**',
       '**/tests/test-files/**',
       '**/tests/test-documents/**',
       '**/react-app-env.d.ts',

package.json

@@ -7,26 +7,24 @@
     "node": ">= 16.0.0"
   },
   "scripts": {
-    "prebuild": "rimraf dist/ .bob/ tsconfig.tsbuildinfo && pnpm prepare-cli-version",
+    "prebuild": "rimraf dist/ .bob/ tsconfig.tsbuildinfo",
     "build": "bob build",
-    "postbuild": "pnpm fix-bins",
+    "postbuild": "pnpm fix-bins && pnpm inject-cli-version",
     "clean": "rimraf node_modules/",
     "dev-test:generate": "pnpm --filter=\"dev-test*\" generate",
     "dev-test:generate:cjs": "pnpm --filter=\"dev-test*\" generate:cjs",
     "dev-test:generate:esm": "pnpm --filter=\"dev-test*\" generate:esm",
-    "dev-test:test-cli-version:cjs": "pnpm prepare-cli-version && pnpm --filter=\"dev-test-general\" test-cli-version:cjs",
-    "dev-test:test-cli-version:esm": "pnpm prepare-cli-version && pnpm --filter=\"dev-test-general\" test-cli-version:esm",
     "dev-test:watch": "pnpm --filter=\"dev-test*\" watch",
     "dev-test:watch:cjs": "pnpm --filter=\"dev-test*\" watch:cjs",
     "dev-test:watch:esm": "pnpm --filter=\"dev-test*\" watch:esm",
     "examples:build": "pnpm --filter=\"example-*\" build",
     "examples:codegen": "pnpm --filter=\"example-*\" codegen",
     "examples:test:end2end": "pnpm --filter=\"example-*\" --workspace-concurrency=1 test:end2end",
     "fix-bins": "node scripts/fix-bin.js",
+    "inject-cli-version": "node scripts/inject-cli-version.mjs",
     "postinstall": "husky install",
     "lint": "eslint --cache .",
     "lint:ci": "eslint --cache --output-file eslint_report.json --format json .",
-    "prepare-cli-version": "node ./scripts/prepare-cli-version.mjs",
     "prettier": "prettier --cache --write --list-different .",
     "prettier:check": "prettier --cache --check .",
     "rebuild": "bob build --incremental",

packages/graphql-codegen-cli/src/config.ts

@@ -17,14 +17,14 @@ import {
   Types,
 } from '@graphql-codegen/plugin-helpers';
 import type { UnnormalizedTypeDefPointer } from '@graphql-tools/load';
-import { version } from './_version.js'; // Run `pnpm prepare-cli-version` to initialize this value
 import { findAndLoadGraphQLConfig } from './graphql-config.js';
 import {
   defaultDocumentsLoadOptions,
   defaultSchemaLoadOptions,
   loadDocuments,
   loadSchema,
 } from './load.js';
+import { version } from './version.js';
 
 const { lstat } = promises;
 

packages/graphql-codegen-cli/src/version.ts

@@ -0,0 +1 @@
+export const version = '__VERSION__';

scripts/prepare-cli-version.mjs scripts/inject-cli-version.mjsscripts/prepare-cli-version.mjs renamed to scripts/inject-cli-version.mjs

@@ -1,5 +1,6 @@
-import * as fs from 'fs';
-import packageJson from '../packages/graphql-codegen-cli/package.json' with { type: 'json' };
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import * as url from 'node:url';
 
 /**
  * This script extracts the `@graphql-codegen/cli` version ahead of build and publish time,
@@ -16,8 +17,28 @@ import packageJson from '../packages/graphql-codegen-cli/package.json' with { ty
  * 5. Use `nodenext` instead of `esnext` for `module` and `moduleResolution`. Maybe a big change?
  * 6. Ship ESM-only. This is a big change.
  */
-fs.writeFileSync(
-  './packages/graphql-codegen-cli/src/_version.ts',
-  `export const version = '${packageJson.version}';`,
-  'utf8',
-);
+
+const __dirname = url.fileURLToPath(new url.URL('.', import.meta.url));
+
+const packageJsonFile = path.resolve(__dirname, '../packages/graphql-codegen-cli/package.json');
+const versionFiles = [
+  path.resolve(__dirname, '../packages/graphql-codegen-cli/dist/cjs/version.js'),
+  path.resolve(__dirname, '../packages/graphql-codegen-cli/dist/esm/version.js'),
+];
+
+const packageJson = JSON.parse(fs.readFileSync(packageJsonFile, 'utf8'));
+
+for (const versionFile of versionFiles) {
+  const versionFileContent = fs.readFileSync(versionFile, 'utf8');
+  fs.writeFileSync(
+    versionFile,
+    versionFileContent.replace('__VERSION__', packageJson.version || 'unknown'),
+    'utf8',
+  );
+
+  const updatedVersionContent = fs.readFileSync(versionFile, 'utf8');
+
+  console.log('***');
+  console.log(`Updated ${versionFile} content:\n"${updatedVersionContent}"`);
+  console.log('***');
+}