Merge pull request #59 from studiometa/bugfix/gitlab-reference

[Bugix] Fix parsing of `!reference` tag

Author: titouanmathis

CHANGELOG.md

@@ -6,6 +6,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ## [Unreleased]
 
+### Fixed
+
+- Fix parsing of !reference tag ([#50](https://github.com/studiometa/prettier-formatter-gitlab/issues/50), [#59](https://github.com/studiometa/prettier-formatter-gitlab/pull/59), [a937d55](https://github.com/studiometa/prettier-formatter-gitlab/commit/a937d55))
+
 ## v2.1.0 - 2024.12.12
 
 ### Added

package-lock.json

@@ -48,8 +48,8 @@
   "dependencies": {
     "chalk": "^5.3.0",
     "jest-diff": "^29.7.0",
-    "js-yaml": "^4.1.0",
-    "prettier-linter-helpers": "^1.0.0"
+    "prettier-linter-helpers": "^1.0.0",
+    "yaml": "2.7.0"
   },
   "peerDependencies": {
     "prettier": "^3.0"

package.json

@@ -1,44 +1,19 @@
 #!/usr/bin/env node
-import { readFileSync, mkdirSync, writeFileSync } from 'node:fs';
-import { cwd, env } from 'node:process';
-import { join, resolve, dirname } from 'node:path';
-import yaml from 'js-yaml';
+import { mkdirSync, writeFileSync } from 'node:fs';
+import { env } from 'node:process';
+import { dirname } from 'node:path';
 import { diff } from './formatters/diff.js';
 import { gitlab } from './formatters/gitlab.js';
 import { parse } from './utils/parse-prettier-results.js';
-
-const {
-  // Used as a fallback for local testing.
-  CI_CONFIG_PATH = '.gitlab-ci.yml',
-  CI_JOB_NAME,
-  CI_PROJECT_DIR = cwd(),
-} = env;
-
-/**
- * Get the output path for the report file.
- * @returns {string}
- */
-function getOutputPath() {
-  const jobs = yaml.load(readFileSync(join(CI_PROJECT_DIR, CI_CONFIG_PATH), 'utf-8'));
-  const { artifacts } = jobs[CI_JOB_NAME];
-  const location = artifacts && artifacts.reports && artifacts.reports.codequality;
-  const msg = `Expected ${CI_JOB_NAME}.artifacts.reports.codequality to be one exact path`;
-  if (!location) {
-    throw new Error(`${msg}, but no value was found.`);
-  }
-  if (Array.isArray(location)) {
-    throw new Error(`${msg}, but found an array instead.`);
-  }
-  return resolve(CI_PROJECT_DIR, location);
-}
+import { getOutputPath } from './utils/get-output-path.js';
 
 /**
  * Format Prettier results for GitLab Code Quality Reports.
  * @param   {string} results
  * @returns {Promise<void>}
  */
 export async function prettierFormatterGitLab(results) {
-  const { PRETTIER_CODE_QUALITY_REPORT } = env;
+  const { CI_JOB_NAME, PRETTIER_CODE_QUALITY_REPORT } = env;
   if (CI_JOB_NAME || PRETTIER_CODE_QUALITY_REPORT) {
     const files = parse(results);
 

src/index.js

@@ -0,0 +1,52 @@
+import { readFileSync, existsSync, lstatSync } from 'node:fs';
+import { cwd, env } from 'node:process';
+import { join, resolve } from 'node:path';
+import { parseDocument } from 'yaml';
+
+/** @type {yaml.CollectionTag} */
+const referenceTag = {
+  tag: '!reference',
+  collection: 'seq',
+  default: false,
+  resolve() {
+    // We only allow the syntax. We don’t actually resolve the reference.
+  },
+};
+
+/**
+ * Get the output path for the report file.
+ * @returns {string}
+ */
+export function getOutputPath() {
+  const {
+    // Used as a fallback for local testing.
+    CI_CONFIG_PATH = '.gitlab-ci.yml',
+    CI_JOB_NAME,
+    CI_PROJECT_DIR = cwd(),
+  } = env;
+
+  const configPath = join(CI_PROJECT_DIR, CI_CONFIG_PATH);
+
+  if (!existsSync(configPath) || !lstatSync(configPath).isFile()) {
+    throw new Error(
+      'Could not resolve .gitlab-ci.yml to automatically detect report artifact path.' +
+        ' Please manually provide a path via the ESLINT_CODE_QUALITY_REPORT variable.',
+    );
+  }
+
+  const doc = parseDocument(readFileSync(configPath, 'utf-8'), {
+    version: '1.1',
+    customTags: [referenceTag],
+  });
+
+  const path = [CI_JOB_NAME, 'artifacts', 'reports', 'codequality'];
+  const location = doc.getIn(path);
+
+  if (typeof location !== 'string' || !location) {
+    throw new TypeError(
+      `Expected ${path.join('.')} to be one exact path, got: ${JSON.stringify(location)}`,
+    );
+  }
+
+  return resolve(CI_PROJECT_DIR, location);
+}

src/utils/get-output-path.js

@@ -2,6 +2,34 @@
 
 exports[`prettier-formatter-gitlab cli should create a code quality report file with prettier -l 1`] = `
 [
+  {
+    "check_name": "prettier",
+    "description": "Delete ⏎",
+    "fingerprint": "994d9053d8e5f73c098f2bcfadec786c",
+    "location": {
+      "lines": {
+        "begin": 11,
+        "end": 12,
+      },
+      "path": "test/__stubs__/.gitlab-ci.fail.yml",
+    },
+    "severity": "minor",
+    "type": "issue",
+  },
+  {
+    "check_name": "prettier",
+    "description": "Delete ⏎",
+    "fingerprint": "20c9ca526d0a43e3075a3e0f74135991",
+    "location": {
+      "lines": {
+        "begin": 11,
+        "end": 12,
+      },
+      "path": "test/__stubs__/.gitlab-ci.yml",
+    },
+    "severity": "minor",
+    "type": "issue",
+  },
   {
     "check_name": "prettier",
     "description": "Replace ·arg·· with arg",
@@ -82,6 +110,34 @@ exports[`prettier-formatter-gitlab cli should create a code quality report file
 
 exports[`prettier-formatter-gitlab cli should create a code quality report file with prettier -c 1`] = `
 [
+  {
+    "check_name": "prettier",
+    "description": "Delete ⏎",
+    "fingerprint": "994d9053d8e5f73c098f2bcfadec786c",
+    "location": {
+      "lines": {
+        "begin": 11,
+        "end": 12,
+      },
+      "path": "test/__stubs__/.gitlab-ci.fail.yml",
+    },
+    "severity": "minor",
+    "type": "issue",
+  },
+  {
+    "check_name": "prettier",
+    "description": "Delete ⏎",
+    "fingerprint": "20c9ca526d0a43e3075a3e0f74135991",
+    "location": {
+      "lines": {
+        "begin": 11,
+        "end": 12,
+      },
+      "path": "test/__stubs__/.gitlab-ci.yml",
+    },
+    "severity": "minor",
+    "type": "issue",
+  },
   {
     "check_name": "prettier",
     "description": "Replace ·arg·· with arg",

test/__snapshots__/cli.spec.js.snap

@@ -0,0 +1,11 @@
+.ref:
+  script:
+    - npm run lint
+
+prettier:
+  script:
+    - !reference [.ref, script]
+  artifacts:
+    reports:
+      codequalit: gl-codequality.json
+

test/__stubs__/.gitlab-ci.fail.yml

@@ -0,0 +1,11 @@
+.ref:
+  script:
+    - npm run lint
+
+prettier:
+  script:
+    - !reference [.ref, script]
+  artifacts:
+    reports:
+      codequality: gl-codequality.json
+

test/__stubs__/.gitlab-ci.yml

@@ -0,0 +1,36 @@
+import { describe, it, expect, beforeEach, afterEach } from 'bun:test';
+import { resolve, join } from 'node:path';
+import { getOutputPath } from '../../src/utils/get-output-path.js';
+
+let env;
+
+beforeEach(() => {
+  env = process.env;
+});
+
+afterEach(() => {
+  process.env = env;
+});
+
+describe('The getPrettierFileInfos function', () => {
+  it('should return the output path defined in a .gitlab-ci.yml file', () => {
+    process.env.CI_PROJECT_DIR = resolve(join(import.meta.dirname, '../__stubs__/'));
+    process.env.CI_JOB_NAME = 'prettier';
+    console.log(process.env.CI_PROJECT_DIR);
+    expect(getOutputPath()).toBe(
+      resolve(join(import.meta.dirname, '../__stubs__/gl-codequality.json')),
+    );
+  });
+
+  it('should throw an error if it can not find the report file path', () => {
+    process.env.CI_PROJECT_DIR = resolve(join(import.meta.dirname, '../__stubs__/'));
+    process.env.CI_JOB_NAME = 'prettier';
+    process.env.CI_CONFIG_PATH = '.gitlab-ci.fail.yml';
+    expect(getOutputPath).toThrow();
+  });
+
+  it('should throw an error if it can not find a .gitlab-ci.yml file', () => {
+    process.env.CI_PROJECT_DIR = '/tmp';
+    expect(getOutputPath).toThrow();
+  });
+});