Merge pull request #19 from alexraputa/prettier

Add Prettier setup and format repository sources

Author: NullVoxPopuli

.editorconfig

@@ -0,0 +1,16 @@
+# EditorConfig helps developers define and maintain consistent
+# coding styles between different editors and IDEs
+# editorconfig.org
+
+root = true
+
+[*]
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+indent_style = space
+indent_size = 2
+
+[*.{diff,md}]
+trim_trailing_whitespace = false

.github/workflows/ember-best-practices-ci.yml

@@ -14,12 +14,10 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - uses: pnpm/action-setup@v2
-        with:
-          version: 10.24.0
       - uses: actions/setup-node@v4
         with:
-          node-version: '20'
-          cache: 'pnpm'
+          node-version-file: package.json
+          cache: pnpm
           cache-dependency-path: pnpm-lock.yaml
       - run: pnpm install
       - run: pnpm --filter ember-best-practices-build validate

.github/workflows/skills-lint.yml

@@ -7,6 +7,21 @@ on:
     branches: [main]
 
 jobs:
+  format:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+    steps:
+      - uses: actions/checkout@v6
+      - uses: pnpm/action-setup@v4
+      - uses: actions/setup-node@v6
+        with:
+          node-version-file: package.json
+          cache: pnpm
+          cache-dependency-path: pnpm-lock.yaml
+      - run: pnpm install
+      - run: pnpm lint:format
+
   lint:
     runs-on: ubuntu-latest
     permissions:
@@ -16,8 +31,8 @@ jobs:
       - uses: pnpm/action-setup@v4
       - uses: actions/setup-node@v6
         with:
-          node-version: '24'
-          cache: 'pnpm'
+          node-version-file: package.json
+          cache: pnpm
           cache-dependency-path: pnpm-lock.yaml
       - run: pnpm install
       - run: pnpm --filter skills-lint test

.prettierignore

@@ -0,0 +1,13 @@
+# Generated skill artifacts
+skills/*/AGENTS.md
+skills/*.zip
+
+# Generated package artifacts
+packages/ember-best-practices-build/test-cases.json
+packages/ember-best-practices-agents-md-codemod/best-practices/
+
+# Temporary output
+**/tmp/
+
+# Lockfiles
+pnpm-lock.yaml

.vscode/settings.json

@@ -7,5 +7,13 @@
         "disabled": false
       }
     }
+  },
+  // ============================================================================
+  // Save and formatting behavior
+  // ============================================================================
+  "editor.formatOnSave": true,
+  "editor.defaultFormatter": "esbenp.prettier-vscode",
+  "[jsonc]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
   }
 }

AGENTS.md

@@ -30,8 +30,13 @@ skills/
 
 ````markdown
 ---
-name: {skill-name}
-description: {One sentence describing when to use this skill. Include trigger phrases like "Deploy my app", "Check logs", etc.}
+name: { skill-name }
+description:
+  {
+    One sentence describing when to use this skill. Include trigger phrases like "Deploy my app",
+    'Check logs',
+    etc.,
+  }
 ---
 
 # {Skill Title}

package.json

@@ -1,8 +1,17 @@
 {
   "name": "agent-skills",
   "private": true,
-  "packageManager": "pnpm@10.24.0",
+  "type": "module",
+  "packageManager": "pnpm@10.32.1",
   "engines": {
     "node": "24.x"
+  },
+  "scripts": {
+    "format": "prettier --write .",
+    "lint:format": "prettier --check ."
+  },
+  "devDependencies": {
+    "prettier": "^3.8.1",
+    "prettier-plugin-ember-template-tag": "^2.1.3"
   }
 }

packages/ember-best-practices-agents-md-codemod/bin/cli.js

@@ -7,16 +7,16 @@
  *   npx ember-best-practices-agents-md --output AGENTS.md
  */
 
-import fs from "fs";
-import path from "path";
+import fs from 'fs';
+import path from 'path';
 import {
   DOCS_DIR_NAME,
   collectDocFiles,
   copyBestPractices,
   ensureGitignoreEntry,
   generateIndex,
   injectIntoAgentsMd,
-} from "../lib/agents-md.js";
+} from '../lib/agents-md.js';
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
@@ -38,18 +38,18 @@ function bold(s) {
 
 async function run(options = {}) {
   const cwd = process.cwd();
-  const targetFile = options.output || "AGENTS.md";
+  const targetFile = options.output || 'AGENTS.md';
   const claudeMdPath = path.join(cwd, targetFile);
   const docsPath = path.join(cwd, DOCS_DIR_NAME);
   const docsLinkPath = `./${DOCS_DIR_NAME}`;
 
   let sizeBefore = 0;
   let isNewFile = true;
-  let existingContent = "";
+  let existingContent = '';
 
   if (fs.existsSync(claudeMdPath)) {
-    existingContent = fs.readFileSync(claudeMdPath, "utf-8");
-    sizeBefore = Buffer.byteLength(existingContent, "utf-8");
+    existingContent = fs.readFileSync(claudeMdPath, 'utf-8');
+    sizeBefore = Buffer.byteLength(existingContent, 'utf-8');
     isNewFile = false;
   }
 
@@ -58,11 +58,11 @@ async function run(options = {}) {
   try {
     copyBestPractices(docsPath);
   } catch (err) {
-    console.error("Error:", err.message);
+    console.error('Error:', err.message);
     process.exit(1);
   }
 
-  const docFiles = collectDocFiles(path.join(docsPath, "rules"));
+  const docFiles = collectDocFiles(path.join(docsPath, 'rules'));
   const filePaths = docFiles.map((f) => `rules/${f.relativePath}`);
 
   const indexContent = generateIndex({
@@ -72,30 +72,29 @@ async function run(options = {}) {
   });
 
   const newContent = injectIntoAgentsMd(existingContent, indexContent);
-  fs.writeFileSync(claudeMdPath, newContent, "utf-8");
+  fs.writeFileSync(claudeMdPath, newContent, 'utf-8');
 
-  const sizeAfter = Buffer.byteLength(newContent, "utf-8");
+  const sizeAfter = Buffer.byteLength(newContent, 'utf-8');
   const gitignoreResult = ensureGitignoreEntry(cwd);
 
-  const action = isNewFile ? "Created" : "Updated";
+  const action = isNewFile ? 'Created' : 'Updated';
   const sizeInfo = isNewFile
     ? formatSize(sizeAfter)
     : `${formatSize(sizeBefore)} → ${formatSize(sizeAfter)}`;
 
-  console.log(`${green("✓")} ${action} ${bold(targetFile)} (${sizeInfo})`);
+  console.log(`${green('✓')} ${action} ${bold(targetFile)} (${sizeInfo})`);
   if (gitignoreResult.updated) {
-    console.log(`${green("✓")} Added ${bold(DOCS_DIR_NAME)} to .gitignore`);
+    console.log(`${green('✓')} Added ${bold(DOCS_DIR_NAME)} to .gitignore`);
   }
-  console.log("");
+  console.log('');
 }
 
 // Parse args (minimal: --output)
 const args = process.argv.slice(2);
-const outputIdx = args.indexOf("--output");
-const output =
-  outputIdx >= 0 && args[outputIdx + 1] ? args[outputIdx + 1] : null;
+const outputIdx = args.indexOf('--output');
+const output = outputIdx >= 0 && args[outputIdx + 1] ? args[outputIdx + 1] : null;
 
-if (args.includes("--help") || args.includes("-h")) {
+if (args.includes('--help') || args.includes('-h')) {
   console.log(`
 ember-best-practices-agents-md - Ember.js best practices for AI coding agents