guard-my-design-system 1.0.2 → 1.0.3

package/README.md

@@ -83,7 +83,7 @@ Judge your uncommitted work before anyone else sees it:
 npx guard-my-design-system@latest
 ```
 
-| Command | What you get |
+| Command                                                   | What you get |
 |---|---|
 | <code>npx&nbsp;guard-my-design-system@latest</code> | Your working tree's added lines judged against main, in the terminal |
 | <code>npx&nbsp;guard-my-design-system@latest&nbsp;&lt;path&gt;</code> | Judge a different repo than the current directory |

package/index.mjs

@@ -13,6 +13,8 @@
  * --strict, 2 could not run.
  */
 import { resolve } from 'node:path';
+import { readFileSync } from 'node:fs';
+import { execFileSync } from 'node:child_process';
 import { learnSystem } from 'roast-my-design-system/engine';
 import { defaultBase, addedLines } from './src/diff.mjs';
 import { judge } from './src/judge.mjs';
@@ -25,7 +27,22 @@ const opt = (name) => {
   return i > -1 && argv[i + 1] ? argv[i + 1] : null;
 };
 
+if (flag('version') || argv.includes('-v')) {
+  const pkg = JSON.parse(readFileSync(new URL('./package.json', import.meta.url), 'utf8'));
+  console.log(`guard-my-design-system ${pkg.version}`);
+  process.exit(0);
+}
+
 const cwd = resolve(argv[0] && !argv[0].startsWith('--') ? argv[0] : '.');
+
+// The guard judges diffs, so a git repository is its whole world. Say so
+// calmly instead of letting git's own fatals leak through.
+try {
+  execFileSync('git', ['rev-parse', '--is-inside-work-tree'], { cwd, stdio: 'ignore' });
+} catch {
+  console.error('guard: this is not a git repository, and the guard judges diffs. Run it inside a repo.');
+  process.exit(2);
+}
 const base = opt('base') ?? defaultBase(cwd);
 if (!base) {
   console.error('guard: cannot find a base to diff against (no origin/main, origin/master, main or master). Pass one with --base <ref>.');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "guard-my-design-system",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Your design system dies one pull request at a time. This makes sure it doesn't. A guard that judges only the lines a change adds, against the system the repo already has, and names the on-system value the author probably meant.",
   "type": "module",
   "bin": {

package/src/diff.mjs

@@ -11,7 +11,7 @@ import { join } from 'node:path';
 export function defaultBase(cwd) {
   for (const ref of ['origin/main', 'origin/master', 'main', 'master']) {
     try {
-      return execFileSync('git', ['merge-base', 'HEAD', ref], { cwd, encoding: 'utf8' }).trim();
+      return execFileSync('git', ['merge-base', 'HEAD', ref], { cwd, encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] }).trim();
     } catch { /* try the next spelling */ }
   }
   return null;