fix lint

Author: fey

__tests__/index.test.js

@@ -1,9 +1,9 @@
-import fs from 'fs';
-import path from 'path';
-import os from 'os';
+import { execSync } from 'node:child_process';
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+import { fileURLToPath, URL } from 'node:url';
 import nock from 'nock';
-import { URL, fileURLToPath } from 'url';
-import { execSync } from 'child_process';
 import { runTests } from '../src/index.js';
 import buildRoutes from '../src/routes.js';
 
@@ -26,18 +26,18 @@ it('runTests', async () => {
       language: 'ruby',
     },
   };
-  nock(url.origin)
-    .get(url.pathname)
-    .query(true)
-    .reply(200, result);
+  nock(url.origin).get(url.pathname).query(true).reply(200, result);
 
   const tmp = os.tmpdir();
   const mountPath = await fsp.mkdtemp(path.join(tmp, 'hexlet-project-'));
   const projectPath = await fsp.mkdtemp(path.join(tmp, 'hexlet-project-'));
   execSync(`cp -r ${projectFixture}/. ${projectPath}`);
 
   await runTests({
-    mountPath, projectPath, verbose: true, projectMemberId,
+    mountPath,
+    projectPath,
+    verbose: true,
+    projectMemberId,
   });
 
   expect(true).toBe(true);

__tests__/packageChecker.test.js

@@ -1,16 +1,12 @@
-import path from 'path';
-import { fileURLToPath } from 'url';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
 
 import checkPackageName from '../src/packageChecker.js';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-const verifiableProjects = [
-  'javascript',
-  'php',
-  'python',
-];
+const verifiableProjects = ['javascript', 'php', 'python'];
 
 // NOTE: Some projects are not packages.
 // Also, in some languages, verification is not needed,
@@ -24,33 +20,31 @@ const notVerifiableProjects = [
   undefined,
 ];
 
-const getFixturePath = (dirname) => (
-  path.join(__dirname, '..', '__fixtures__', 'package_files', dirname)
-);
+const getFixturePath = (dirname) =>
+  path.join(__dirname, '..', '__fixtures__', 'package_files', dirname);
 
 describe('test projects with correct package name', () => {
   const codePath = getFixturePath('correct');
 
   test.each(verifiableProjects)('%s', (sourceLang) => {
-    expect(() => checkPackageName(codePath, sourceLang))
-      .not.toThrow();
+    expect(() => checkPackageName(codePath, sourceLang)).not.toThrow();
   });
 });
 
 describe('test projects with wrong package name', () => {
   const codePath = getFixturePath('wrong');
 
   test.each(verifiableProjects)('%s', (sourceLang) => {
-    expect(() => checkPackageName(codePath, sourceLang))
-      .toThrow(/^Package name should be .+ instead of wrong-package-name$/);
+    expect(() => checkPackageName(codePath, sourceLang)).toThrow(
+      /^Package name should be .+ instead of wrong-package-name$/,
+    );
   });
 });
 
 describe('test not verifiable projects', () => {
   const codePath = getFixturePath('correct');
 
   test.each(notVerifiableProjects)('%s', (sourceLang) => {
-    expect(() => checkPackageName(codePath, sourceLang))
-      .not.toThrow();
+    expect(() => checkPackageName(codePath, sourceLang)).not.toThrow();
   });
 });

bin/run-post-actions.js

@@ -12,7 +12,9 @@ const mountPath = core.getInput('mount-path', { required: true });
 const projectMemberId = core.getInput('hexlet-id', { required: true });
 
 const params = {
-  mountPath, projectMemberId, verbose,
+  mountPath,
+  projectMemberId,
+  verbose,
 };
 
 try {

bin/run-tests.js

@@ -2,7 +2,7 @@
 
 // @ts-check
 
-import path from 'path';
+import path from 'node:path';
 import * as core from '@actions/core';
 import cleanStack from 'clean-stack';
 
@@ -15,19 +15,27 @@ core.debug(process.cwd());
 
 const mountPath = core.getInput('mount-path', { required: true });
 const verbose = core.getBooleanInput('verbose', { required: false });
-const projectPath = path.resolve(process.cwd(), process.env.ACTION_PROJECT_PATH || '');
+const projectPath = path.resolve(
+  process.cwd(),
+  process.env.ACTION_PROJECT_PATH || '',
+);
 const projectMemberId = core.getInput('hexlet-id', { required: true });
 
 core.exportVariable('PWD', path.join(mountPath, 'source'));
 
 const params = {
-  projectPath, mountPath, verbose, projectMemberId,
+  projectPath,
+  mountPath,
+  verbose,
+  projectMemberId,
 };
 
 try {
   await runTests(params);
 } catch (e) {
-  core.error('The tests have failed. Examine what they have to say. Inhale deeply. Exhale. Fix the code.');
+  core.error(
+    'The tests have failed. Examine what they have to say. Inhale deeply. Exhale. Fix the code.',
+  );
   // NOTE: бектрейс экшена пользователям не нужен
   if (!verbose) {
     e.stack = cleanStack(e.stack);

biome.json

@@ -0,0 +1,60 @@
+{
+  "$schema": "./node_modules/@biomejs/biome/configuration_schema.json",
+  "vcs": {
+    "enabled": true,
+    "clientKind": "git",
+    "useIgnoreFile": true
+  },
+  "files": {
+    "ignoreUnknown": false,
+    "includes": [
+      "**",
+      "!**/*.json",
+      "!**/*.css",
+      "!**/*.scss",
+      "!test",
+      "!public",
+      "!app/assets/builds",
+      "!app/javascript/routes.*",
+      "!app/javascript/types/serializers",
+      "!dist"
+    ]
+  },
+  "formatter": {
+    "enabled": true,
+    "indentStyle": "space"
+  },
+  "linter": {
+    "enabled": true,
+    "rules": {
+      "recommended": true,
+      "complexity": {
+        "noBannedTypes": "off"
+      },
+      "correctness": {
+        "noUnusedImports": "off",
+        "noUnusedFunctionParameters": "off",
+        "noUnusedVariables": "off"
+      },
+      "suspicious": {
+        "noExplicitAny": "off"
+      },
+      "style": {
+        "noNonNullAssertion": "off"
+      }
+    }
+  },
+  "javascript": {
+    "formatter": {
+      "quoteStyle": "single"
+    }
+  },
+  "assist": {
+    "enabled": true,
+    "actions": {
+      "source": {
+        "organizeImports": "on"
+      }
+    }
+  }
+}

server.js

@@ -5,16 +5,19 @@ import Fastify from 'fastify';
 const fastify = Fastify({ logger: true });
 
 // Declare a route
-fastify.get('/api/user_project_github_workflow/project_members/:id', async () => {
-  const result = {
-    tests_on: true,
-    project: {
-      image_name: 'hexlet-project-source-ci',
-    },
-  };
+fastify.get(
+  '/api/user_project_github_workflow/project_members/:id',
+  async () => {
+    const result = {
+      tests_on: true,
+      project: {
+        image_name: 'hexlet-project-source-ci',
+      },
+    };
 
-  return result;
-});
+    return result;
+  },
+);
 
 // Run the server!
 const start = async () => {

src/index.js

@@ -3,19 +3,18 @@
 // https://github.com/actions/javascript-action
 // https://github.com/actions/toolkit/blob/master/docs/action-debugging.md
 
-import fs from 'fs';
-import path from 'path';
+import fs from 'node:fs';
+import path from 'node:path';
 import { DefaultArtifactClient } from '@actions/artifact';
 import * as core from '@actions/core';
-import * as io from '@actions/io';
 import * as exec from '@actions/exec';
+import * as glob from '@actions/glob';
 import { HttpClient } from '@actions/http-client';
+import * as io from '@actions/io';
 import colors from 'ansi-colors';
 import yaml from 'js-yaml';
-import * as glob from '@actions/glob';
-
-import buildRoutes from './routes.js';
 import checkPackageName from './packageChecker.js';
+import buildRoutes from './routes.js';
 
 const uploadArtifacts = async (diffpath) => {
   if (!fs.existsSync(diffpath)) {
@@ -59,9 +58,9 @@ const uploadTestData = async (options) => {
     return;
   }
 
-  const existPaths = artifacts.filter((artifactPath) => (
-    fs.existsSync(path.join(projectSourcePath, artifactPath))
-  ));
+  const existPaths = artifacts.filter((artifactPath) =>
+    fs.existsSync(path.join(projectSourcePath, artifactPath)),
+  );
 
   if (existPaths.length === 0) {
     return;
@@ -75,7 +74,11 @@ const uploadTestData = async (options) => {
   const artifactName = 'test-data';
   const artifactClient = new DefaultArtifactClient();
   const archivePath = path.join(projectSourcePath, archiveName);
-  await artifactClient.uploadArtifact(artifactName, [archivePath], projectSourcePath);
+  await artifactClient.uploadArtifact(
+    artifactName,
+    [archivePath],
+    projectSourcePath,
+  );
   core.info(colors.bgYellow.black('Download snapshots from Artifacts.'));
 };
 
@@ -101,7 +104,10 @@ const prepareProject = async (options) => {
   await exec.exec(copyCmd, null, cmdOptions);
   await io.mkdirP(codePath);
   await io.cp(`${projectPath}/.`, codePath, { recursive: true });
-  await exec.exec('docker', ['build', '--cache-from', projectImageName, '.'], { ...cmdOptions, cwd: projectSourcePath });
+  await exec.exec('docker', ['build', '--cache-from', projectImageName, '.'], {
+    ...cmdOptions,
+    cwd: projectSourcePath,
+  });
 };
 
 const check = async ({ projectSourcePath, codePath, projectMember }) => {
@@ -110,7 +116,11 @@ const check = async ({ projectSourcePath, codePath, projectMember }) => {
   const options = { cwd: projectSourcePath };
   // NOTE: Installing dependencies is part of testing the project.
   await exec.exec('docker compose', ['run', 'app', 'make', 'setup'], options);
-  await exec.exec('docker compose', ['-f', 'docker-compose.yml', 'up', '--abort-on-container-exit'], options);
+  await exec.exec(
+    'docker compose',
+    ['-f', 'docker-compose.yml', 'up', '--abort-on-container-exit'],
+    options,
+  );
 
   const checkState = {
     state: 'success',
@@ -166,12 +176,7 @@ export const runPostActions = async (params) => {
   const { mountPath, projectMemberId, verbose } = params;
   const projectSourcePath = path.join(mountPath, 'source');
 
-  const diffpath = path.join(
-    mountPath,
-    'source',
-    'tmp',
-    'artifacts',
-  );
+  const diffpath = path.join(mountPath, 'source', 'tmp', 'artifacts');
 
   const options = {
     projectSourcePath,

src/packageChecker.js

@@ -1,8 +1,9 @@
 // @ts-check
 
-import fs from 'fs';
-import path from 'path';
+import fs from 'node:fs';
+import path from 'node:path';
 import ini from 'ini';
+
 // import yaml from 'js-yaml';
 // import _ from 'lodash';
 
@@ -15,7 +16,8 @@ const parsers = {
 const getFullPath = (dirpath, filename) => path.resolve(dirpath, filename);
 const getFormat = (filepath) => path.extname(filepath).slice(1);
 const parse = (content, format) => parsers[format](content);
-const getData = (filepath) => parse(fs.readFileSync(filepath, 'utf-8'), getFormat(filepath));
+const getData = (filepath) =>
+  parse(fs.readFileSync(filepath, 'utf-8'), getFormat(filepath));
 
 const mapping = {
   python: {
@@ -28,15 +30,13 @@ const mapping = {
   },
   php: {
     expectedPackageName: 'hexlet/code',
-    getPackageName: (codePath) => (
-      getData(getFullPath(codePath, 'composer.json')).name
-    ),
+    getPackageName: (codePath) =>
+      getData(getFullPath(codePath, 'composer.json')).name,
   },
   javascript: {
     expectedPackageName: '@hexlet/code',
-    getPackageName: (codePath) => (
-      getData(getFullPath(codePath, 'package.json')).name
-    ),
+    getPackageName: (codePath) =>
+      getData(getFullPath(codePath, 'package.json')).name,
   },
 };
 
@@ -53,7 +53,9 @@ const checkPackageName = (codePath, sourceLang) => {
   const packageName = getPackageName(codePath);
 
   if (packageName !== expectedPackageName) {
-    throw new Error(`Package name should be ${expectedPackageName} instead of ${packageName}`);
+    throw new Error(
+      `Package name should be ${expectedPackageName} instead of ${packageName}`,
+    );
   }
 };
 

src/routes.js

@@ -1,5 +1,5 @@
 // TODO https://hexlet.io/api/user-project-github-workflow/projects/:slug/
-import path from 'path';
+import path from 'node:path';
 
 const apiUrl = '/api/user_project_github_workflow/';