chore(harness): add Plan-by footer, source Acked-by/Plan-by from opencode.json

Every conventional commit now includes three footers: Plan-by (sourced
from agent.plan.model in opencode.json), Acked-by (sourced from
agent.build.model), and Signed-off-by. Both model IDs use the segment
after the last "/" — e.g. openrouter/z-ai/glm-5.2 becomes glm-5.2.

Acked-by is no longer the runtime model; it now reads the configured
build model from opencode.json, making both footers stable references.

commitlint enforces all three via a custom trailers-exist plugin rule
(replaces the built-in trailer-exists which only accepts a single
string). Updated the conventional-commits skill, tdd agent,
writing-plans skill, finishing-a-development-branch skill, AGENTS.md,
README.md, and commitlint.config.js.

Plan-by: glm-5.2
Acked-by: deepseek-v4-pro
Signed-off-by: kyau <git@kyaulabs.com>

Author: kyau

.opencode/agents/tdd.md

@@ -142,7 +142,8 @@ and produce a commit message in the required format:
 - Type and optional scope from the work performed (feat, fix, test, docs,
   chore, etc.)
 - Subject: lowercase, no period, ≤ 100 chars, describes what changed
-- Footer: `Acked-by:` with the current model ID in kebab-case
+- Footer: `Plan-by:` with `agent.plan.model` from `opencode.json`, segment after the last `/` (e.g. `openrouter/z-ai/glm-5.2` → `glm-5.2`)
+- Footer: `Acked-by:` with `agent.build.model` from `opencode.json`, segment after the last `/` (e.g. `deepseek/deepseek-v4-pro` → `deepseek-v4-pro`)
 - Footer: `Signed-off-by: kyau <git@kyaulabs.com>`
 
 If the task already provided a commit message in the plan, validate it —

.opencode/skills/conventional-commits/SKILL.md

@@ -16,26 +16,30 @@ description: Use when writing or reviewing commit messages. Covers the required
 - Subject line: lowercase, no period at end, max 100 characters
 - Body: wrap at 72 characters, explain *why* not *what*
 - Signed commits required (`git commit -S`)
-- Every commit must include `Acked-by:` (agent in kebab-case) and `Signed-off-by:` (user) footers
+- Every commit must include `Plan-by:`, `Acked-by:`, and `Signed-off-by:` footers
 
 ## Required Footers
 
-Every commit message must end with two footers:
+Every commit message must end with three footers:
 
-- **`Acked-by:`** — the AI agent that authored the change, in kebab-case.
-Use the model ID without slashes. Examples: `deepseek-v4-pro`,
-`claude-sonnet-4`, `gemini-2-5-pro`.
+- **`Plan-by:`** — the planning model, in kebab-case. Sourced from
+  `agent.plan.model` in `opencode.json` (the segment after the last `/`).
+  Example: `openrouter/z-ai/glm-5.2` → `glm-5.2`.
+- **`Acked-by:`** — the build model, in kebab-case. Sourced from
+  `agent.build.model` in `opencode.json` (the segment after the last `/`).
+  Example: `deepseek/deepseek-v4-pro` → `deepseek-v4-pro`.
 
 > [!CAUTION]
-> Do NOT use role names (`build-agent`, `code-review`, `tdd`, etc.) —
-> only the model ID. The Acked-by tracks which model produced the
-> commit, not which agent role orchestrated it.
+> Do NOT use role names (`build-agent`, `code-review`, `tdd`, etc.) — only the
+> model ID. The Plan-by and Acked-by footers track which configured models
+> planned and built the change, not which agent role orchestrated it.
 - **`Signed-off-by:`** — the human user approving the change, formatted as
   `username <email>`. Default when no user is specified:
   `kyau <git@kyaulabs.com>`.
 
-These are mandatory for traceability. The agent writes them automatically based
-on the current model and the AGENTS.md default.
+These are mandatory for traceability. The agent writes them automatically by
+reading `agent.plan.model` and `agent.build.model` from `opencode.json` and
+taking the segment after the last `/`.
 
 ## Valid Types
 
@@ -79,13 +83,15 @@ directory, or feature area: `feat(aurora)`, `fix(db)`, `test(auth)`.
 ```
 feat(auth): add remember-me cookie to login flow
 
+Plan-by: glm-5.2
 Acked-by: deepseek-v4-pro
 Signed-off-by: kyau <git@kyaulabs.com>
 ```
 
 ```
 fix(db): prevent SQL injection in user search query
 
+Plan-by: glm-5.2
 Acked-by: deepseek-v4-pro
 Signed-off-by: kyau <git@kyaulabs.com>
 Fixes: #42
@@ -94,6 +100,7 @@ Fixes: #42
 ```
 test(auth): add boundary cases for empty credentials
 
+Plan-by: glm-5.2
 Acked-by: deepseek-v4-pro
 Signed-off-by: kyau <git@kyaulabs.com>
 ```

.opencode/skills/finishing-a-development-branch/SKILL.md

@@ -29,7 +29,7 @@ Run every item; do not skip:
 - [ ] No merge conflicts after rebase. If conflicts exist, suggest
       `@resolve-merge-conflicts` or manual resolution.
 - [ ] All commits follow Conventional Commits format and include
-      `Acked-by:` + `Signed-off-by:` footers (see `conventional-commits` skill).
+      `Plan-by:` + `Acked-by:` + `Signed-off-by:` footers (see `conventional-commits` skill).
 
 After every item passes, present the summary below. If any item fails, stop
 and fix it — do not proceed with a failing item.

.opencode/skills/writing-plans/SKILL.md

@@ -146,6 +146,7 @@ Expected: PASS
 git add tests/path/to/Test.php backend/path/to/file.php
 git commit -S -m "feat(scope): concise subject describing the change
 
+Plan-by: glm-5.2
 Acked-by: deepseek-v4-pro
 Signed-off-by: kyau <git@kyaulabs.com>"
 ```
@@ -165,7 +166,7 @@ failures** — never write them:
   code steps)
 - References to types, functions, or methods not defined in any task
 - Bare commit messages missing scope or required footers — use the full
-  conventional-commits format (type[scope]: subject + Acked-by + Signed-off-by)
+  conventional-commits format (type[scope]: subject + Plan-by + Acked-by + Signed-off-by)
 
 ## Self-review
 

AGENTS.md

@@ -134,14 +134,15 @@ For linting details and responsive/mobile-first CSS rules, see `scss-mobile-firs
 - Features: `feat/<username>-<hash>-<description>`
 - Commits: Conventional Commits format (type[scope]: subject) — see `conventional-commits` skill
 - Signed commits required
-- Every commit must include `Acked-by:` (model ID in kebab-case, e.g. `deepseek-v4-pro`) and `Signed-off-by:` (user) footers. Default Signed-off-by: `kyau <git@kyaulabs.com>`.
+- Every commit must include `Plan-by:` (sourced from `agent.plan.model` in `opencode.json`), `Acked-by:` (sourced from `agent.build.model` in `opencode.json` — model ID segment after the last `/`), and `Signed-off-by:` (user) footers. Default Signed-off-by: `kyau <git@kyaulabs.com>`.
 - No squash merges. Each logical change is its own atomic commit — the git history serves as the development and evaluation log. A pre-push hook warns on single-commit branches that look like squashes.
 
 After implementing any change — whether via @tdd, a direct fix, an issue
 tracker resolution, or a fast-path trivial change — produce a commit message
 in conventional commits format before committing. Load the
-`conventional-commits` skill and produce: type[scope]: subject + Acked-by +
-Signed-off-by footers. The commit-msg hook blocks invalid messages, but the
+`conventional-commits` skill and produce: type[scope]: subject + Plan-by +
+Acked-by + Signed-off-by footers. The commit-msg hook blocks invalid messages,
+but the
 message should be well-formed before you reach the hook.
 
 ### Commit and push permissions

README.md

@@ -580,7 +580,8 @@ Longer commit body with additional contextual information about the code changes
 <token>: <value>
 (max-length: 100)
 token (Sentence-case) = {
-  'Acked-by',           # Required — the AI agent in kebab-case (e.g., deepseek-v4-pro)
+  'Plan-by',            # Required — the planning model from opencode.json (e.g., glm-5.2)
+  'Acked-by',           # Required — the build model from opencode.json (e.g., deepseek-v4-pro)
   'Signed-off-by',      # Required — the user (e.g., kyau <git@kyaulabs.com>)
   'BREAKING CHANGE',    # Required when the type/scope includes !
   'Cc',
@@ -591,10 +592,11 @@ token (Sentence-case) = {
 }
 ```
 
-Every commit must include `Acked-by` and `Signed-off-by` footers. If no user is
-explicitly named, the default `Signed-off-by` is `kyau <git@kyaulabs.com>`. The
-`Acked-by` value is the agent that authored the change in kebab-case (use the
-model ID without slashes, e.g., `deepseek-v4-pro`, `claude-sonnet-4`).
+Every commit must include `Plan-by`, `Acked-by`, and `Signed-off-by` footers. If
+no user is explicitly named, the default `Signed-off-by` is `kyau
+<git@kyaulabs.com>`. `Plan-by` and `Acked-by` are sourced from `agent.plan.model`
+and `agent.build.model` in `opencode.json` (the segment after the last `/`),
+e.g. `deepseek/deepseek-v4-pro` → `deepseek-v4-pro`.
 
 ### Examples
 
@@ -609,6 +611,7 @@ As per #123 recommendation input controller is now based on blah.
 
 Basic movement added.
 
+Plan-by: glm-5.2
 Acked-by: deepseek-v4-pro
 Signed-off-by: kyau <git@kyaulabs.com>
 Refs: #123
@@ -620,6 +623,7 @@ fix: array parsing issue
 
 Fixes: #42
 Cc: Z
+Plan-by: glm-5.2
 Acked-by: deepseek-v4-pro
 Reviewed-by: Z
 Signed-off-by: kyau <git@kyaulabs.com>

commitlint.config.js

@@ -1,5 +1,34 @@
+// $KYAULabs: commitlint.config.js kyau@nova 2026/07/05 -0700 Exp $
+
+const { spawnSync } = require('child_process');
+
+const trailersExist = (parsed, when, trailers) => {
+	const output = spawnSync('git', ['interpret-trailers', '--parse'], {
+		input: parsed.raw || '',
+	}).stdout.toString();
+	const lines = output.split('\n');
+	const negated = when === 'never';
+	const missing = trailers.filter(
+		(t) => !lines.some((ln) => ln.startsWith(t))
+	);
+	const allPresent = missing.length === 0;
+	return [
+		negated ? !allPresent : allPresent,
+		'message must have ' +
+			trailers.map((t) => '`' + t + '`').join(', ') +
+			' trailer' + (trailers.length > 1 ? 's' : ''),
+	];
+};
+
 module.exports = {
 	extends: ['@commitlint/config-conventional'],
+	plugins: [
+		{
+			rules: {
+				'trailers-exist': trailersExist,
+			},
+		},
+	],
 	rules: {
 		'header-max-length': [2, 'always', 100],
 		'type-enum': [2, 'always', [
@@ -17,7 +46,9 @@ module.exports = {
 			'test',
 			'ignore',
 		]],
-		'trailer-exists': [2, 'always', 'Acked-by:'],
+		'trailers-exist': [2, 'always', ['Plan-by:', 'Acked-by:']],
 		'signed-off-by': [2, 'always', 'Signed-off-by:'],
 	},
 };
+
+// vim: ft=javascript sts=4 sw=4 ts=4 noet :