chore: fill derivable placeholders, drop false ARCHITECTURE, surface the rest

[hyperpolymath]

Automated estate top-up. Nothing here invents a value.

Filled — every token with one mechanical answer (owner, repo, forge, author, dates, project name, main branch). Identity from the git remote, dates from the clock, name from the README H1.

Not filled, on purpose — SECURITY_EMAIL (two competing addresses exist in the estate), RESPONSE_TIME, CONDUCT_TEAM (substitutes into "a {{CONDUCT_TEAM}} member", which is not English), WEBSITE, PROJECT_DESCRIPTION, LANG_STACK. More than one defensible answer exists, and a confident wrong value is worse than a visible gap.

Deleted — ARCHITECTURE.md where it is byte-identical to the 346-copy estate boilerplate (blob 607e3d8c). Those 33 lines describe a src/ tests/ docs/ scripts/ config/ tree this repo does not have. Matched by hash, so a genuinely written ARCHITECTURE can never be caught by it.

CODEOWNERS — the solo form from standards/CODEOWNERS-POLICY.adoc Rule 1, which forbids a catch-all where the only owner is the sole maintainer. templates/CODEOWNERS contradicts that policy; the policy is versioned, dated and resolves standards#55, so it wins. Genuine co-owners (Rule 2) are untouched.

Surfaced — REQUIRES_INITIALISATION.md plus a priority action in 0-AI-MANIFEST.a2ml, listing every remaining token, what it means, which files it belongs in, why it was not done already, and that it is to be deleted only when the work is genuinely complete.

Built from a fresh clone of origin/main, never a local checkout — several of those are dirty and hold unpushed commits.

🤖 Generated with Claude Code

[gitar-bot]

Note

Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime.
Learn more

Code Review ⚠️ Changes requested 0 resolved / 1 findings

Automated estate top-up replacing placeholders and cleaning up boilerplate, but incorrectly flags Justfile's {{DB_URL}} as an unfilled token and corrupts a debt note in the guard logic.

⚠️ Bug: REQUIRES_INITIALISATION flags Justfile's {{DB_URL}} which is not a placeholder

📄 REQUIRES_INITIALISATION.md:5 📄 REQUIRES_INITIALISATION.md:42-47 📄 0-AI-MANIFEST.a2ml:106-120 📄 .machine_readable/bot_directives/methodology.a2ml:105 📄 .machine_readable/6a2/STATE.a2ml:60

The tool mistook Just's own variable-interpolation syntax for an unfilled template token: DB_URL is already defined with a real value at Justfile:5 (DB_URL := "postgresql://..."), and the {{DB_URL}} occurrences in the recipes are just referencing that variable — not template placeholders awaiting a value. As a result the entire new REQUIRES_INITIALISATION.md and the blocking 'PRIORITY ACTION' added to 0-AI-MANIFEST.a2ml are built on a false positive; they tell every future agent/person the repo is 'not fully initialised' when its only listed outstanding token is already resolved. Exclude .just/Justfile files from the placeholder scan (or only treat {{FOO}} as a token when FOO is not a defined Just variable), then drop REQUIRES_INITIALISATION.md and the manifest priority section since there is nothing outstanding.

Teach the estate top-up scanner that {{DB_URL}} is a defined Just variable, so REQUIRES_INITIALISATION.md / the manifest priority action are not generated.

// Placeholder scanner: skip Just variable interpolation.
// {{VAR}} in a Justfile is a variable reference, not a template token.
// A token is only outstanding when VAR is NOT assigned via `VAR := ...`.
// DB_URL is defined at Justfile:5, so it must not be reported here.

🤖 Prompt for agents

Code Review: Automated estate top-up replacing placeholders and cleaning up boilerplate, but incorrectly flags Justfile's {{DB_URL}} as an unfilled token and corrupts a debt note in the guard logic.

1. ⚠️ Bug: REQUIRES_INITIALISATION flags Justfile's {{DB_URL}} which is not a placeholder
   Files: REQUIRES_INITIALISATION.md:5, REQUIRES_INITIALISATION.md:42-47, 0-AI-MANIFEST.a2ml:106-120, .machine_readable/bot_directives/methodology.a2ml:105, .machine_readable/6a2/STATE.a2ml:60

   The tool mistook Just's own variable-interpolation syntax for an unfilled template token: `DB_URL` is already defined with a real value at Justfile:5 (`DB_URL := "postgresql://..."`), and the `{{DB_URL}}` occurrences in the recipes are just referencing that variable — not template placeholders awaiting a value. As a result the entire new REQUIRES_INITIALISATION.md and the blocking 'PRIORITY ACTION' added to 0-AI-MANIFEST.a2ml are built on a false positive; they tell every future agent/person the repo is 'not fully initialised' when its only listed outstanding token is already resolved. Exclude `.just`/`Justfile` files from the placeholder scan (or only treat `{{FOO}}` as a token when `FOO` is not a defined Just variable), then drop REQUIRES_INITIALISATION.md and the manifest priority section since there is nothing outstanding.

   Fix (Teach the estate top-up scanner that {{DB_URL}} is a defined Just variable, so REQUIRES_INITIALISATION.md / the manifest priority action are not generated.):
   // Placeholder scanner: skip Just variable interpolation.
   // {{VAR}} in a Justfile is a variable reference, not a template token.
   // A token is only outstanding when VAR is NOT assigned via `VAR := ...`.
   // DB_URL is defined at Justfile:5, so it must not be reported here.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact

gitar display:verbose

---

Important

Your trial ends in 5 days — upgrade now to keep code review, CI analysis, auto-apply, custom automations, and more.

_{Was this helpful? React with 👍 / 👎 | Gitar}

[gitar-bot]

⚠️ Bug: REQUIRES_INITIALISATION flags Justfile's {{DB_URL}} which is not a placeholder

The tool mistook Just's own variable-interpolation syntax for an unfilled template token: DB_URL is already defined with a real value at Justfile:5 (DB_URL := "postgresql://..."), and the {{DB_URL}} occurrences in the recipes are just referencing that variable — not template placeholders awaiting a value. As a result the entire new REQUIRES_INITIALISATION.md and the blocking 'PRIORITY ACTION' added to 0-AI-MANIFEST.a2ml are built on a false positive; they tell every future agent/person the repo is 'not fully initialised' when its only listed outstanding token is already resolved. Exclude .just/Justfile files from the placeholder scan (or only treat {{FOO}} as a token when FOO is not a defined Just variable), then drop REQUIRES_INITIALISATION.md and the manifest priority section since there is nothing outstanding.

Teach the estate top-up scanner that {{DB_URL}} is a defined Just variable, so REQUIRES_INITIALISATION.md / the manifest priority action are not generated.:

// Placeholder scanner: skip Just variable interpolation.
// {{VAR}} in a Justfile is a variable reference, not a template token.
// A token is only outstanding when VAR is NOT assigned via `VAR := ...`.
// DB_URL is defined at Justfile:5, so it must not be reported here.

_{Was this helpful? React with 👍 / 👎}

[gitar-bot]

⚠️ This PR is blocked due to unresolved code review findings.

_{Configure merge blocking · Maintainers can dismiss this review.}

[hyperpolymath]

Closing in favour of a reworked substituter — hyperpolymath/standards#590.

This sweep filled {{TOKEN}} placeholders by plain text replacement across every file. Across the 90 repositories reviewed it drew 141 findings, and one is severe:

sed "s/{{PROJECT_NAME}}/$name/g"  →  sed "s/Conative Gating/$name/g"
sed -e "s/{{DATE}}/$DATE/g"       →  sed -e "s/2026-08-05/$DATE/g"

Those are the scripts whose job is to perform template substitution. Filling the left-hand side of their own sed expressions means template application silently stops working — and the breakage stays invisible until someone mints a repository from the template and gets a half-substituted tree.

Four further shapes came out of the same cause:

• release.sh no longer substitutes {{DATE}} into release notes
• just's own {{ARGS}} reported as an unfilled token, leaving repos permanently "not initialised"
• instructional docs rewritten to "Replace laminar, laminar, {{DEPS}} with actuals"
• documentation describing placeholders had its examples filled in

The rule plain replacement cannot express: a placeholder is sometimes a value to fill and sometimes the subject being discussed. standards#590 encodes that distinction and is tested against this exact corruption (9/9, the first three cases reproducing it).

Nothing here is lost — the placeholder filling will be redone with that tool. Closing rather than fixing forward because repairing 289 branches individually would repeat the mistake at the same scale.