@yourfam/yf-commit 0.1.1 → 0.2.1

package/LICENSE

@@ -1,16 +1,16 @@
-MIT License
+Copyright (c) 2026 YourFam. All rights reserved.
 
-Copyright (c) 2026 YourFam
+This software is proprietary and confidential to YourFam.
 
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
+You may install and use the yf-commit command as distributed by YourFam
+(for example via npm as @yourfam/yf-commit).
 
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
+You may not copy, modify, merge, publish, distribute, sublicense, sell,
+or create derivative works of this software, in source or compiled form,
+except as expressly permitted in writing by YourFam.
+
+No license is granted under any patent, trademark, or other right except
+the limited right to use the command as stated above.
 
 THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,

package/README.md

@@ -24,7 +24,7 @@ Then from any git repo:
 yf-commit
 ```
 
-The command is `yf-commit`. After a **global** install, npm prints a short hint (`yf-commit installed…`). That does not run for `-D`, CI, or `npx`.
+The command is `yf-commit`.
 
 ## Alternate modes of install
 
@@ -120,10 +120,28 @@ yf-commit --all -y     # add everything → commit now (no confirm)
 yf-commit --print      # print only; do not commit (still needs a staged diff, or add --all)
 yf-commit --all --print
 yf-commit --type fix   # force type
-yf-commit --type 4     # same (1–6: chore, docs, feat, fix, refactor, test)
+yf-commit --type 4     # same (1–11)
 yf-commit --all -y --type feat
 ```
 
+The JS does not classify the **type**. It does measure **size** (insertions + deletions, plus file count) and tells the model a length budget: tiny diffs get exactly 1 WHY + 1 WHAT; large diffs get more WHAT, grouped. WHY does not grow with every extra hundred lines. Mixed diffs still get **one** dominant type (`feat` / `fix` usually). Override with `--type`. Default still prints the draft so you can cancel.
+
+| # | Type | Meaning |
+|---|---|---|
+| 1 | `chore` | Maintenance that is none of the below |
+| 2 | `docs` | Docs only |
+| 3 | `feat` | User-visible new capability |
+| 4 | `fix` | User-visible bug |
+| 5 | `refactor` | Same behavior, different structure |
+| 6 | `test` | Tests only |
+| 7 | `build` | Compile, bundler, packaging |
+| 8 | `ci` | Pipeline / GitHub Actions / hooks |
+| 9 | `perf` | Same behavior, measurably faster |
+| 10 | `style` | Formatting only, zero logic |
+| 11 | `revert` | Undoes a previous commit |
+
+Priority on mixed diffs: `revert` → `fix` → `feat` → `test` → `docs` → `perf` → `refactor` → `style` → `ci` → `build` → `chore`.
+
 Default prints the draft in the terminal. Enter or `y` to commit, `n` or Ctrl+C to cancel. It does not open an editor.
 
 Nothing staged → error, exit 1. No `git add` unless `--all`.
@@ -158,8 +176,10 @@ npm test
 node ./bin/yf-commit.js --help
 ```
 
-Maintainer: YourFam (`kamal-yourfam` on npm). License MIT.
+Maintainer: YourFam (`kamal-yourfam` on npm). Proprietary. All rights reserved.
 
 ## License
 
-MIT
+Proprietary. Copyright (c) 2026 YourFam. All rights reserved.
+
+You may install and run `yf-commit` as distributed by YourFam. You may not copy, modify, or redistribute the source. See [LICENSE](./LICENSE).

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@yourfam/yf-commit",
-  "version": "0.1.1",
+  "version": "0.2.1",
   "description": "AI commit message from your staged git diff",
   "type": "module",
   "bin": {
-    "yf-commit": "./bin/yf-commit.js"
+    "yf-commit": "bin/yf-commit.js"
   },
   "files": [
     "bin",
@@ -16,7 +16,6 @@
     "node": ">=20"
   },
   "scripts": {
-    "postinstall": "node ./src/postinstall-run.js",
     "test": "node --test test/*.test.js",
     "yf-commit": "node ./bin/yf-commit.js"
   },
@@ -31,7 +30,7 @@
     "email": "kamal@yourfam.co",
     "url": "https://github.com/YourFam"
   },
-  "license": "MIT",
+  "license": "UNLICENSED",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/YourFam/yf-commit.git"

package/src/args.js

@@ -1,5 +1,5 @@
 import { CliError } from "./errors.js";
-import { parseType } from "./types.js";
+import { parseType, typeHelpLines } from "./types.js";
 
 /**
  * @typedef {object} ParsedArgs
@@ -58,7 +58,7 @@ export function parseArgs(argv) {
     } else if (a === "--type") {
       const value = args[i + 1];
       if (value == null || value.startsWith("-")) {
-        throw new CliError("Flag --type requires a value: feat|fix|… or 1–6.");
+        throw new CliError("Flag --type requires a value: feat|fix|… or 1–11.");
       }
       result.type = parseType(value);
       i += 1;
@@ -96,11 +96,14 @@ Usage:
   yf-commit -y           Commit immediately (also --yes, --auto)
   yf-commit --print      Print message only; do not commit
   yf-commit --all        git add -A, then same as default
-  yf-commit --type fix   Force type (name or 1–6)
+  yf-commit --type fix   Force type (name or 1–11)
   yf-commit init         Provider, model, API key → ~/.yf-commit/config.json
   yf-commit init --show  Masked status
   yf-commit init --reset Delete saved config
 
+Types (mixed diffs: one dominant type, not a blend):
+${typeHelpLines()}
+
 This does not run your test suite. YourFam does not give you an API key.
 
 Need a host we don't list? Set YF_COMMIT_BASE_URL and YF_COMMIT_MODEL.`;

package/src/diff-profile.js

@@ -0,0 +1,114 @@
+const ORDER = ["tiny", "small", "medium", "large", "xl"];
+
+export const PROFILES = {
+  tiny: {
+    id: "tiny",
+    why: "exactly 1 bullet",
+    what: "exactly 1 bullet",
+    groupWhat: false,
+    maxTokens: 400,
+  },
+  small: {
+    id: "small",
+    why: "1–2 bullets",
+    what: "2–3 bullets",
+    groupWhat: false,
+    maxTokens: 700,
+  },
+  medium: {
+    id: "medium",
+    why: "2–3 bullets",
+    what: "4–6 bullets",
+    groupWhat: false,
+    maxTokens: 1200,
+  },
+  large: {
+    id: "large",
+    why: "2–3 bullets",
+    what: "6–10 bullets, grouped by area if needed",
+    groupWhat: true,
+    maxTokens: 1800,
+  },
+  xl: {
+    id: "xl",
+    why: "2–4 bullets (do not grow WHY with every file)",
+    what: "8–14 bullets, grouped by area",
+    groupWhat: true,
+    maxTokens: 2500,
+  },
+};
+
+/**
+ * Count changed lines (+ and −, excluding +++ / --- headers) and files.
+ * @param {string} diff
+ */
+export function countDiff(diff) {
+  const lines = String(diff).split(/\r?\n/);
+  let loc = 0;
+  const files = new Set();
+
+  for (const line of lines) {
+    if (line.startsWith("diff --git ")) {
+      const m = line.match(/^diff --git a\/(.+) b\/(.+)$/);
+      if (m) {
+        const path = m[2] === "/dev/null" ? m[1] : m[2];
+        files.add(path.replace(/^"|"$/g, ""));
+      }
+      continue;
+    }
+    if (line.startsWith("Binary files ")) {
+      loc += 1;
+      continue;
+    }
+    if (line.startsWith("+") && !line.startsWith("+++")) {
+      loc += 1;
+      continue;
+    }
+    if (line.startsWith("-") && !line.startsWith("---")) {
+      loc += 1;
+    }
+  }
+
+  return { loc, files: files.size };
+}
+
+function profileIdFromLoc(loc) {
+  if (loc <= 20) return "tiny";
+  if (loc <= 80) return "small";
+  if (loc <= 300) return "medium";
+  if (loc <= 800) return "large";
+  return "xl";
+}
+
+function applyFileFloor(id, fileCount) {
+  let i = ORDER.indexOf(id);
+  if (fileCount >= 8) i = Math.max(i, ORDER.indexOf("small"));
+  if (fileCount >= 20) i = Math.max(i, ORDER.indexOf("medium"));
+  return ORDER[i];
+}
+
+/**
+ * @param {string} diff
+ */
+export function diffProfile(diff) {
+  const { loc, files } = countDiff(diff);
+  const id = applyFileFloor(profileIdFromLoc(loc), files);
+  const spec = PROFILES[id];
+  return { ...spec, loc, files };
+}
+
+/**
+ * @param {ReturnType<typeof diffProfile>} profile
+ */
+export function lengthInstruction(profile) {
+  const fileNote =
+    profile.files >= 8
+      ? "FILES IMPACTED: list real paths; group under area headings if that helps. Never write \"Updated N files.\""
+      : "FILES IMPACTED: list each real path. Never write \"Updated N files.\"";
+
+  return `Diff size: ${profile.loc} line(s), ${profile.files} file(s). Profile: ${profile.id}.
+WHY: ${profile.why}. Reasons only — do not recap every file.
+WHAT CHANGED: ${profile.what}.
+${fileNote}
+Do not add extra bullets to look complete. Do not pad.`;
+}

package/src/index.js

@@ -24,6 +24,7 @@ import { completeChat } from "./llm.js";
 import { readFileSync } from "node:fs";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
+import { diffProfile, lengthInstruction } from "./diff-profile.js";
 import { buildSystemPrompt, buildUserPrompt } from "./prompt.js";
 
 const pkg = JSON.parse(
@@ -74,12 +75,14 @@ export async function main(argv) {
     }
   }
 
+  const profile = diffProfile(diff);
   const message = await completeChat({
     baseUrl: settings.baseUrl,
     model: settings.resolvedModel,
     apiKey: settings.apiKey || "ollama",
     systemPrompt: buildSystemPrompt({ forcedType: args.type }),
-    userPrompt: buildUserPrompt(diff),
+    userPrompt: buildUserPrompt(diff, lengthInstruction(profile)),
+    maxTokens: profile.maxTokens,
   });
 
   if (args.print) {

package/src/llm.js

@@ -15,6 +15,7 @@ export async function completeChat({
   apiKey,
   systemPrompt,
   userPrompt,
+  maxTokens = 1500,
 }) {
   const root = String(baseUrl || "").replace(/\/+$/, "");
   const url = `${root}/chat/completions`;
@@ -34,7 +35,7 @@ export async function completeChat({
       body: JSON.stringify({
         model,
         temperature: 0.3,
-        max_tokens: 1500,
+        max_tokens: maxTokens,
         messages: [
           { role: "system", content: systemPrompt },
           { role: "user", content: userPrompt },

package/src/prompt.js

@@ -1,11 +1,20 @@
-import { TYPE_EMOJI } from "./types.js";
+import {
+  TYPE_DEFS,
+  TYPE_EMOJI,
+  TYPE_PRIORITY,
+  TYPES,
+} from "./types.js";
 
 const MAX_DIFF_CHARS = 100_000;
 
+const EMOJI_LINE = TYPES.map((t) => `${TYPE_EMOJI[t]} ${t}`).join(", ");
+
+const DEF_LINES = TYPES.map((t) => `- ${t}: ${TYPE_DEFS[t]}`).join("\n");
+
 export function buildSystemPrompt({ forcedType } = {}) {
   const typeLine = forcedType
     ? `You MUST use type "${forcedType}" and emoji ${TYPE_EMOJI[forcedType]} on line 1.`
-    : "Use conventional commit types: feat, fix, refactor, chore, docs, test.";
+    : `Use exactly one conventional commit type: ${TYPES.join(", ")}.`;
 
   return `You are a commit message generator. Output only the commit message. No preamble, no markdown fences, no extra commentary.
 
@@ -24,22 +33,26 @@ emoji type(scope): brief description
 
 Rules:
 - ${typeLine}
-- Emoji on line 1 (✨ feat, 🐛 fix, ♻️ refactor, 🔧 chore, 📝 docs, 🧪 test).
+- Type meanings:
+${DEF_LINES}
+- ${TYPE_PRIORITY}
+- Emoji on line 1 (${EMOJI_LINE}).
 - First line under 72 characters.
 - WHY: user/business impact, plain language.
 - WHAT CHANGED: technical, from the diff, not staging stats.
 - FILES IMPACTED: paths relative to this repository's root (the current working directory).
-- Adaptive length: tiny diffs → few bullets; large diffs → more, grouped if needed. Never pad. Never write "Updated N staged file(s)."
+- Obey the length profile in the user message (bullet counts for WHY / WHAT). Never pad. Never write "Updated N staged file(s)."
 - No footer. Do not add WORKTREE, BRANCH, MACHINE, or hostname lines.
 - Do not mention any product or company unless it appears in the diff.`;
 }
 
-export function buildUserPrompt(diff) {
+export function buildUserPrompt(diff, lengthBlock = "") {
   let body = String(diff);
   if (body.length > MAX_DIFF_CHARS) {
     body = `${body.slice(0, MAX_DIFF_CHARS)}\n\n[diff truncated]`;
   }
-  return `Generate a commit message for this staged git diff:\n\n${body}`;
+  const prefix = lengthBlock ? `${lengthBlock}\n\n` : "";
+  return `${prefix}Generate a commit message for this staged git diff:\n\n${body}`;
 }
 
 export function cleanMessage(text) {

package/src/types.js

@@ -1,7 +1,22 @@
 import { CliError } from "./errors.js";
 
-/** Alphabetical 1–6 map: chore, docs, feat, fix, refactor, test */
-export const TYPES = ["chore", "docs", "feat", "fix", "refactor", "test"];
+/**
+ * 1–6 stay stable (chore, docs, feat, fix, refactor, test).
+ * 7–11 are the extra conventional types.
+ */
+export const TYPES = [
+  "chore",
+  "docs",
+  "feat",
+  "fix",
+  "refactor",
+  "test",
+  "build",
+  "ci",
+  "perf",
+  "style",
+  "revert",
+];
 
 export const TYPE_EMOJI = {
   feat: "✨",
@@ -10,8 +25,42 @@ export const TYPE_EMOJI = {
   chore: "🔧",
   docs: "📝",
   test: "🧪",
+  build: "📦",
+  ci: "👷",
+  perf: "⚡",
+  style: "🎨",
+  revert: "⏪",
+};
+
+export const TYPE_DEFS = {
+  feat: "User-visible new capability",
+  fix: "User-visible bug",
+  docs: "Documentation only, no product code",
+  test: "Tests only",
+  revert: "Undoes a previous commit",
+  perf: "Same behavior, measurably faster",
+  refactor: "Same behavior, different structure",
+  style: "Formatting only, zero logic",
+  ci: "Pipeline / GitHub Actions / hooks only",
+  build: "Compile, bundler, packaging only",
+  chore: "Maintenance that is none of the above",
 };
 
+/** Mixed diffs: one primary type, not a blend. */
+export const TYPE_PRIORITY = `Pick exactly one type for the whole commit. Mixed diffs are normal — choose the dominant type, in this order:
+1. revert — the change undoes a previous commit
+2. fix — any user-visible bug
+3. feat — any user-visible new capability (tests/docs/formatting in the same diff stay feat)
+4. test — tests only
+5. docs — docs only
+6. perf — same feature, measurably faster (not a bugfix)
+7. refactor — same behavior, different structure (not formatting-only)
+8. style — formatting / lint only, zero logic
+9. ci — CI / GitHub Actions / hooks only
+10. build — compile, bundler, native addons, packaging only
+11. chore — everything else (deps, ignore files, version bump with no product change)
+Do not invent extra types. Do not combine types.`;
+
 /**
  * @param {string | null | undefined} raw
  * @returns {string | null}
@@ -19,9 +68,19 @@ export const TYPE_EMOJI = {
 export function parseType(raw) {
   if (raw == null || raw === "") return null;
   const s = String(raw).trim().toLowerCase();
-  if (/^[1-6]$/.test(s)) return TYPES[Number(s) - 1];
+  if (/^\d+$/.test(s)) {
+    const n = Number(s);
+    if (n >= 1 && n <= TYPES.length) return TYPES[n - 1];
+  }
   if (TYPES.includes(s)) return s;
   throw new CliError(
-    `Unknown type "${raw}". Use feat, fix, refactor, chore, docs, test, or 1–6.`,
+    `Unknown type "${raw}". Use ${TYPES.join(", ")}, or 1–${TYPES.length}.`,
   );
 }
+
+export function typeHelpLines() {
+  return TYPES.map((name, i) => {
+    const n = i + 1;
+    return `  ${n}. ${TYPE_EMOJI[name]} ${name} — ${TYPE_DEFS[name]}`;
+  }).join("\n");
+}

package/src/postinstall-run.js

@@ -1,7 +0,0 @@
-import { runPostinstall } from "./postinstall.js";
-
-try {
-  runPostinstall();
-} catch {
-  // Never fail install.
-}

package/src/postinstall.js

@@ -1,22 +0,0 @@
-/**
- * Print a short hint after `npm install -g` only.
- * Stay silent for -D, CI, npx, and repo `npm install`.
- */
-export function isGlobalNpmInstall(env = process.env) {
-  return env.npm_config_global === "true";
-}
-
-export function installHint() {
-  return `yf-commit installed. Run: yf-commit
-First time: provider → model → API key
-Docs: https://github.com/YourFam/yf-commit`;
-}
-
-export function runPostinstall(
-  env = process.env,
-  write = (s) => process.stderr.write(s),
-) {
-  if (!isGlobalNpmInstall(env)) return false;
-  write(`${installHint()}\n`);
-  return true;
-}