@nt-ai-lab/opencode-skillz 0.3.6 → 0.3.8

package/dist/tools/lint.js

@@ -16,7 +16,7 @@ class GitCommandError extends Error {
         super(message);
     }
 }
-class MissingLocalDependencyError extends Error {
+class MissingBundledLintAssetError extends Error {
     constructor(message) {
         super(message);
     }
@@ -26,9 +26,9 @@ class LintExecutionError extends Error {
         super(message);
     }
 }
+const ansiEscapeSequencePattern = new RegExp(String.raw `\u001B\[[0-?]*[ -/]*[@-~]`, "g");
 const toolDirectory = path.dirname(fileURLToPath(import.meta.url));
 const toolRepositoryRoot = path.resolve(toolDirectory, "..", "..");
-const eslintCliPath = path.join(toolRepositoryRoot, "node_modules", "eslint", "bin", "eslint.js");
 const eslintConfigPath = path.join(toolRepositoryRoot, "scripts", "living-architecture-eslint.config.mjs");
 const gitBinaryPath = "/usr/bin/git";
 export const LINT_TOOL_NAME = "nt_skillz_lint";
@@ -63,11 +63,11 @@ function normalizeFilePaths(filePaths) {
     normalizedFilePaths.forEach(ensureSupportedFilePath);
     return normalizedFilePaths;
 }
-function ensureLocalDependencies() {
-    if (fs.existsSync(eslintCliPath)) {
+function ensureBundledLintAssetsExist() {
+    if (fs.existsSync(eslintConfigPath)) {
         return;
     }
-    throw new MissingLocalDependencyError(`Expected ESLint dependencies to be installed in ${toolRepositoryRoot}.`);
+    throw new MissingBundledLintAssetError(`Expected bundled lint config to exist at ${eslintConfigPath}.`);
 }
 function runGitCommand(repositoryRoot, gitArguments) {
     const gitResult = spawnSync(gitBinaryPath, ["-C", repositoryRoot, ...gitArguments], { encoding: "utf8" });
@@ -131,6 +131,9 @@ function createLintTitle(filePaths, baseReference) {
     }
     return "Lint current TypeScript files";
 }
+function removeAnsiEscapeSequences(value) {
+    return value.replaceAll(ansiEscapeSequencePattern, "");
+}
 async function runEslint(repositoryRoot, lintTargets) {
     const previousLintRepositoryRoot = process.env.NT_SKILLZ_LINT_REPO_ROOT;
     process.env.NT_SKILLZ_LINT_REPO_ROOT = repositoryRoot;
@@ -143,7 +146,7 @@ async function runEslint(repositoryRoot, lintTargets) {
     try {
         const lintResults = await eslint.lintFiles(lintTargets);
         const formatter = await eslint.loadFormatter("stylish");
-        const formattedOutput = (await formatter.format(lintResults)).trim();
+        const formattedOutput = removeAnsiEscapeSequences((await formatter.format(lintResults)).trim());
         const errorCount = lintResults.reduce((count, lintResult) => count + lintResult.errorCount + lintResult.fatalErrorCount, 0);
         return {
             exitCode: errorCount > 0 ? 1 : 0,
@@ -192,7 +195,7 @@ function parsePortableLintCommandLine(commandLineArguments) {
     };
 }
 export async function runPortableLint(request) {
-    ensureLocalDependencies();
+    ensureBundledLintAssetsExist();
     const repositoryRoot = resolveDirectory(request.repositoryRoot ?? process.cwd());
     const baseReference = normalizeOptionalText(request.base);
     const headReference = normalizeOptionalText(request.head);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nt-ai-lab/opencode-skillz",
-  "version": "0.3.6",
+  "version": "0.3.8",
   "description": "Bundled OpenCode commands and agents",
   "type": "module",
   "main": "dist/index.js",

package/scripts/living-architecture-eslint.config.mjs

@@ -22,8 +22,6 @@ if (!lintRepositoryRoot) {
 
 const typescriptFiles = ['**/*.ts', '**/*.tsx']
 const testFiles = ['**/*.spec.ts', '**/*.spec.tsx', '**/*.test.ts', '**/*.test.tsx']
-const thinLayerFiles = ['**/entrypoint/**/*.ts', '**/commands/**/*.ts', '**/queries/**/*.ts']
-const thinLayerIgnoredFiles = ['**/*.spec.ts', '**/*.test.ts']
 const ignoredPaths = [
   '**/dist',
   '**/out-tsc',
@@ -58,21 +56,6 @@ const noEmptyStringFallbackRule = {
 
 const restrictedSyntaxRules = ['error', noLetRule, noGenericErrorRule, noEmptyStringFallbackRule]
 
-const entrypointRestrictedSyntaxRules = [
-  'error',
-  noLetRule,
-  noGenericErrorRule,
-  {
-    selector: 'FunctionDeclaration:not([parent.type="ExportNamedDeclaration"])',
-    message: 'Entrypoints must not define private functions. Move logic to commands/, queries/, or infra/.',
-  },
-  {
-    selector: 'VariableDeclarator > ArrowFunctionExpression',
-    message: 'Entrypoints must not define private arrow functions. Move logic to commands/, queries/, or infra/.',
-  },
-  noEmptyStringFallbackRule,
-]
-
 const restrictedImportPatterns = [
   {
     group: ['*/utils/*', '*/utils', '*/utilities'],
@@ -229,20 +212,6 @@ export default tseslint.config(
       '@stylistic/object-property-newline': ['error', { allowAllPropertiesOnSameLine: false }],
     },
   },
-  {
-    files: thinLayerFiles,
-    ignores: thinLayerIgnoredFiles,
-    rules: {
-      'max-lines': ['error', { max: 150, skipBlankLines: true, skipComments: true }],
-    },
-  },
-  {
-    files: ['**/entrypoint/**/*.ts'],
-    ignores: thinLayerIgnoredFiles,
-    rules: {
-      'no-restricted-syntax': entrypointRestrictedSyntaxRules,
-    },
-  },
   {
     files: typescriptFiles,
     plugins: {