Skip to content

drain(#448 follow-up): resolve _patterns.md surface-class contradiction #466

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
AceHack merged 2 commits into from
Apr 25, 2026
Merged

drain(#448 follow-up): resolve _patterns.md surface-class contradiction #466

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c388b1b
drain(#448 follow-up): resolve _patterns.md surface-class contradiction
AceHack Apr 25, 2026
cb62527
drain(#466 follow-up): fix hyphen-line-wrap + tighten taxonomy glob
AceHack Apr 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
47 changes: 42 additions & 5 deletions docs/pr-preservation/_patterns.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,9 @@ permitted; preserves provenance:
- `docs/DECISIONS/**`
- `docs/ROUND-HISTORY.md`
- `docs/aurora/**`
- `docs/pr-preservation/**`
- `docs/pr-preservation/*-drain-log.md` (per-log records;
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot Apr 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Keep CI-fix logs in the history-class surface

Narrowing docs/pr-preservation/** to docs/pr-preservation/*-drain-log.md drops existing provenance docs like docs/pr-preservation/141-ci-fix-log.md out of history-class, so that file now implicitly falls under current-state rules even though it is the same point-in-time record type (names, checks, and audit trail). This reintroduces taxonomy drift for review tooling and attribution guidance; the history-class selector should include all preservation logs in this directory (or explicitly include the CI-fix log shape).

Useful? React with 👍 / 👎.

`_patterns.md` is the synthesis-over-history exception per
the section below)
- `memory/**` (out-of-scope for code-style rules anyway)

**Current-state-class surfaces** — role-ref discipline applies; no
Expand All @@ -285,6 +287,17 @@ first-name attribution:
- Behavioural docs, threat models
- Spec docs (`docs/**` not in history-class above)

**Synthesis-over-history surfaces** — current-state-tracking
abstractions over history-class corpora; reflect current state
not historical sequence:

- `docs/pr-preservation/_patterns.md` (this file) —
synthesis-index over the per-PR drain-log corpus. Edited as
new logs land + patterns surface; should reflect current
corpus state. Underscore-prefix candidate convention for
"synthesis-indices over history-class corpora" co-located
with the corpus.

**Tension surfaces** — both apply; defer to maintainer:

- `docs/bootstrap/**` (#195) — current-state operational substrate
Expand All @@ -295,7 +308,9 @@ first-name attribution:
The **Otto-279 history-vs-current-state carve-out is mature enough
to codify in `docs/AGENT-BEST-PRACTICES.md` as a stable BP-NN rule.**
Tension surfaces need either a third surface class or explicit
carve-out within current-state.
carve-out within current-state. Synthesis-over-history is now
named as a third class that's distinct from both pure history-
class and pure current-state.

---

Expand Down Expand Up @@ -357,9 +372,31 @@ When a new drain-log lands in `docs/pr-preservation/`:
3. If an outcome class needs renaming or a tension surface gets a
maintainer-decision, update the taxonomy sections.

This file is itself current-state operational substrate: it should
reflect the corpus state, not the historical sequence in which
patterns were discovered.
This file is a **synthesis-index over a history-class corpus** —
the per-PR `*-drain-log.md` files in this directory are the
history-class records (preserve names, point-in-time records,
provenance, Otto-279 carve-out applies), and this `_patterns.md`
is the current-state-tracking abstraction over them. The two
surface classes coexist in the same directory by design:

- `docs/pr-preservation/*-drain-log.md` — **history-class** per-PR
records (per the surface-class taxonomy above; Otto-279 carve-out
applies; first-name attribution preserves provenance).
- `docs/pr-preservation/_patterns.md` (this file) —
**current-state synthesis-index** that should reflect the
corpus state, not the historical sequence in which patterns
were discovered. This is a third surface class —
*synthesis-over-history* — distinct from both pure
history-class and pure current-state surfaces.

The directory-level surface-class declaration in the
"Surface-class taxonomy" section above applies to the per-log
records (the per-log glob `docs/pr-preservation/*-drain-log.md`);
the index file is explicitly carved out as a synthesis-index.
The underscore prefix (`_patterns.md`) is a candidate
naming convention for "synthesis-indices over history-class
corpora" — when the convention matures, it could be promoted to
a named pattern itself.

## Reference patterns

Expand Down
Loading