Skip to content

docs(agents): replace unslop and the review decisions registry - #345

Merged
davidvornholt merged 2 commits into
mainfrom
docs/reduce-agent-context
Sep 9, 2026
Merged

docs(agents): replace unslop and the review decisions registry#345
davidvornholt merged 2 commits into
mainfrom
docs/reduce-agent-context

Conversation

@davidvornholt

@davidvornholt davidvornholt commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Before. The mandatory unslop skill contains 981 words, and reviews load a 3,608-word decisions registry covering unrelated work. The registry repeats rationale already present in code and project instructions, while the consumer template encourages another permanent review log.

Now. Replace unslop with a short plain-writing rule in AGENTS.md and keep documentation focused on helping people use, operate, or change the project. Remove the decisions registry and its template, preserve missing rationale beside the owning implementation or in existing focused instructions.

Implementation

  • Remove unslop from the canonical sync manifest and its obsolete Turbo test input. Existing managed copies are pruned on the next sync.
  • Existing consumer-owned decisions files remain untouched. New consumers no longer receive the template, and review skills no longer require loading a registry.
  • Documentation guidance applies to READMEs and other project documents. Keep documentation concise and current, with local rationale near the code and change history in PRs.

Verification

Completed checks

  • bun run check:fix: all 9 tasks passed before the final prose-only reductions, including the existing managed-file pruning tests.
  • Skill validation passed for review, review-fix, and ci-pipelines. The initial attempt lacked Python's YAML module; rerunning the validator in a temporary Nix Python environment succeeded.
  • Revalidated all three edited skills and passed git diff --check after the final prose-only reductions.
  • Searched tracked guidance, templates, code, and configuration for remaining unslop, decisions.md, and registry references; none remain.

Base automatically changed from docs/trim-agent-instructions to main September 9, 2026 15:39
@davidvornholt
davidvornholt force-pushed the docs/reduce-agent-context branch from 122e9f3 to 73b9b63 Compare September 9, 2026 15:45
@davidvornholt
davidvornholt enabled auto-merge (squash) September 9, 2026 15:50
@davidvornholt
davidvornholt force-pushed the docs/reduce-agent-context branch from 73b9b63 to 96e97ce Compare September 9, 2026 18:40
@davidvornholt
davidvornholt merged commit 343a9d3 into main Sep 9, 2026
5 checks passed
@davidvornholt
davidvornholt deleted the docs/reduce-agent-context branch September 9, 2026 18:42
davidvornholt added a commit to davidvornholt/portfolio that referenced this pull request Sep 9, 2026
**Before.** Repository instructions still included the unslop skill and
a separate review decisions file, and canonical files lagged behind the
latest standards.

**Now.** Sync standards through
[#345](davidvornholt/standards#345) at
`343a9d3`, including concise writing and documentation guidance, and
delete `.agents/review/decisions.md`. The standards CLI remains on the
latest release, `0.26.0`.

## Verification

### Completed checks

- `bun install --frozen-lockfile` — passed with the repository’s
declared Bun version.
- `bun standards check` — passed.
- `git diff --check` — passed.
- Searched for stale `decisions.md` and unslop references — none remain.
- GitHub Actions `quality` and `check` — passed.
davidvornholt added a commit to davidvornholt/papersync that referenced this pull request Sep 9, 2026
**Before.** Repository instructions still included the unslop skill and
a separate review decisions file, and canonical files lagged behind the
latest standards.

**Now.** Sync standards through
[#345](davidvornholt/standards#345) at
`343a9d3`, including concise writing and documentation guidance, and
delete `.agents/review/decisions.md`. The standards CLI remains on the
latest release, `0.26.0`.

## Verification

### Completed checks

- `bun install --frozen-lockfile` — passed with the repository’s
declared Bun version.
- `bun standards check` — passed.
- `git diff --check` — passed.
- Searched for stale `decisions.md` and unslop references — none remain.
- GitHub Actions `quality` and `check` — passed.
davidvornholt added a commit to davidvornholt/rota that referenced this pull request Sep 9, 2026
**Before.** Repository instructions still included the unslop skill and
a separate review decisions file, and canonical files lagged behind the
latest standards.

**Now.** Sync standards through
[#345](davidvornholt/standards#345) at
`343a9d3`, including concise writing and documentation guidance, and
delete `.agents/review/decisions.md`. The standards CLI remains on the
latest release, `0.26.0`.

## Verification

### Completed checks

- `bun install --frozen-lockfile` — passed with the repository’s
declared Bun version.
- `bun standards check` — passed.
- `git diff --check` — passed.
- Searched for stale `decisions.md` and unslop references — none remain.
- GitHub Actions `quality` and `check` — passed.
davidvornholt added a commit to davidvornholt/wordhold that referenced this pull request Sep 9, 2026
**Before.** Repository instructions still included the unslop skill and
a separate review decisions file, and canonical files lagged behind the
latest standards.

**Now.** Sync standards through
[#345](davidvornholt/standards#345) at
`343a9d3`, including concise writing and documentation guidance, and
delete `.agents/review/decisions.md`. The standards CLI remains on the
latest release, `0.26.0`.

## Verification

### Completed checks

- `bun install --frozen-lockfile` — passed with the repository’s
declared Bun version.
- `bun standards check` — passed.
- `git diff --check` — passed.
- Searched for stale `decisions.md` and unslop references — none remain.
- GitHub Actions `quality` and `check` — passed.
davidvornholt added a commit to davidvornholt/postlude that referenced this pull request Sep 9, 2026
**Before.** Repository instructions still included the unslop skill and
a separate review decisions file, and canonical files lagged behind the
latest standards.

**Now.** Sync standards through
[#345](davidvornholt/standards#345) at
`343a9d3`, including concise writing and documentation guidance, and
delete `.agents/review/decisions.md`. The standards CLI remains on the
latest release, `0.26.0`.

## Verification

### Completed checks

- `bun install --frozen-lockfile` — passed with the repository’s
declared Bun version.
- `bun standards check` — passed.
- `git diff --check` — passed.
- Searched for stale `decisions.md` and unslop references — none remain.
- GitHub Actions `quality` and `check` — passed.
davidvornholt added a commit to davidvornholt/punktlandung that referenced this pull request Sep 9, 2026
**Before.** Punktlandung used standards CLI 0.22.2 and older canonical
tooling, with verbose agent guidance and a separate review decisions
file.

**Now.** Sync standards at `343a9d3`, including
[#345](davidvornholt/standards#345), upgrade the
CLI to `0.26.0`, and delete `.agents/review/decisions.md`. Biome and
Playwright now match the synced configuration and accessibility package;
existing tests comply with the stricter lint rules.

## Implementation

- Use Biome `2.5.11` and Playwright `1.62.1`; keep Bun at the declared
`1.3.14`.
- Add the required Turbo error-log option to root checks.
- Use Playwright namespace imports, check that test buttons exist before
accessing their props, label hidden test controls, and apply formatter
changes. Application behavior is unchanged.

## Verification

### Completed checks

- `bun run check:fix` from the root with Bun `1.3.14` — passed all 14
Turbo tasks and the repository-artifact test, including database,
component, and accessibility tests.
- Earlier runs found outdated root scripts, unsupported Biome
configuration, and test lint findings, all fixed. A database run failed
with connection refused before starting an isolated PostgreSQL instance.
One invocation from the app directory found no root `check:fix` script;
reran from the root.
- `bun install --frozen-lockfile` — passed.
- `git diff --check` — passed.
- Searched for stale `decisions.md` and unslop references — none remain.
- GitHub Actions `quality` and `check` — passed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant