memory: author template for absorb-time lint hygiene#167
Merged
Conversation
Quick-reference doc capturing the five markdownlint classes that repeatedly fired during the 2026-04-23 Overlay A cadence (PRs #157 / #158 / #159 / #162 / #164): - MD003 — atx (not setext) heading style; blank line before "---" - MD018 — no space after hash; rephrase to avoid "#NNN" at line start - MD022 — blanks around headings; single-line headings only - MD026 — no trailing punctuation; "## Why" not "## Why:" - MD032 — blanks around lists Wrong/right examples plus self-check checklist. Sits next to memory/README.md as a peer reference. Content-level discipline (frontmatter schema, signal-in-signal- out, newest-first ordering, in-repo-preferred migration) cross- references the canonical sources; not duplicated here. Living doc — update when a sixth absorb-time lint class is observed. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
Closed a pattern-capture loop: the five markdownlint classes that fired across the 2026-04-23 Overlay A migration cadence (MD003 / MD018 / MD022 / MD026 / MD032) now have a quick-reference author template sitting peer-to memory/README.md. Durable template means future Overlay A migrations + fresh memory authors don't re-derive the same cleanups. Living doc; updates on sixth observed class. Self-scheduled free work. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ea7ddf1f6b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
Adds a memory/ authoring template to prevent recurring markdownlint failures during memory-file “absorb-time” edits, by documenting common rule triggers with wrong/right examples.
Changes:
- Adds
memory/MEMORY-AUTHOR-TEMPLATE.mdwith guidance for the five repeatedly-hit markdownlint classes (MD003/MD018/MD022/MD026/MD032). - Includes a self-check checklist and pointers to related memory/documentation surfaces.
…finding Resolves the legitimate Copilot review finding on this PR — memory/README.md conventions say new files under memory/ should be indexed at the top of MEMORY.md. Entry added newest-first. The other two review findings (broken path to memory/feedback_signal_in_signal_out_clean_or_better_dsp_discipline.md) were correct at review-time but are resolved by PR #157 which migrated that memory in-repo. When #157 merges, the path resolves cleanly. Per Aaron 2026-04-23: resolving review threads on disagreement is the intended escape — no branch-protection change needed. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…rsation-resolution clarification Real PR blocker diagnosed: required_conversation_resolution was the gate; Codex + Copilot bot threads were unresolved on every session PR. Exercised unblock flow on PR #167 (fixed the legitimate finding, resolved 3 threads). Auto-merge armed; will complete when CI re-passes. Two Aaron directives absorbed: - Demo breaks should be logged (per-user memory filed) so Phase-2 transition inherits a lessons-learned substrate; focus on API-consumer and state-serialization breaks. - required_conversation_resolution never needs an escape hatch — agent can resolve threads directly with a reply. Simplifies forward branch-protection design. Three wrong blocker theories before the right one; adding "check reviewThreads" to the PR-blocker-triage heuristic. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
deleted the
docs/memory-author-template-absorb-time-lint-classes
branch
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…low validated First session PR landed end-to-end. PR #167 merged at 17:11:56Z validates the full unblock pattern: fix-at-source / resolve-threads / let-auto-merge-fire. PR #158 exercised the disagree-with-rationale branch of the pattern — Copilot suggested cleaner description; disagreed per signal-in-signal-out discipline (verbatim is anchor); replied with rationale; resolved thread; rebased on main (MEMORY.md conflict from #167's merge — kept both entries); force-pushed. Auto-merge armed. 15 PRs remain with 120 total unresolved threads. Serial rebase cascade is the bottleneck at scale. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
Two session PRs now merged (#167, #158). PR #166 unblocked this tick: 3 Copilot findings — "Aaron" → "human maintainer" (fix); memory-path clarified as per-user (fix); row-#54-numbering-is-intentional (reply with rationale). Rebased on advanced main; auto-merge armed. Pattern now exercised across all three branches: agree-fix-at-source, disagree-with-rationale, mixed. Merge cadence ~5 min per PR for small-finding cases. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…rst refinement PR #171 (HB-004) unblocked: 3 Copilot findings addressed (row-ordering, memory-path, "Aaron" → "human maintainer" in HB-004 narrative). Rebased on advanced main; auto-merge armed. PR #172 (Pages-UI BACKLOG row) amended with Aaron's read-only- first refinement: Phase 1 read-only uses public-repo GitHub REST API with no auth; Phase 2 write-actions need session/OAuth or thin backend, breaks git-native constraint, deferred. 3 session PRs merged so far (#167, #158, #166); 2 more armed. Sub-5-min per simple PR; higher-thread PRs will take longer. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
4 session PRs merged (#167, #158, #166, #171). PR #171 landed at 17:25:04Z. Aaron reminder absorbed: bun+TS only for Pages-UI; no Jekyll. PR #172 amended + rebased + force-pushed. Directive-to-PR-amendment compression is high (same-tick turnaround). Pages-UI row is accumulating into a mini-design through sequential refinements; promote-to-research-doc if more arrive. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds
memory/MEMORY-AUTHOR-TEMPLATE.md— a quick-reference doc capturing the five markdownlint classes that repeatedly fired during the 2026-04-23 Overlay A cadence, so future memory authors (human or agent) don't rediscover the same cleanups.The five classes
---separators#NNNat line start## Whynot## Why:Each class has wrong/right examples + a one-line rule. Self-check checklist at the bottom.
Observed from
Six memory migrations under PRs #157 / #158 / #159 / #162 / #164 and the initial #155 fire-history fix. Each migration hit one or more of these classes; the template distills the cleanups into a single page.
Scope
Absorb-time lint hygiene only. Content-level discipline (frontmatter schema, signal-in-signal-out, in-repo-preferred) cross-references canonical sources rather than duplicating.
Living doc
Update when a sixth class is observed consistently across multiple drafts.
🤖 Generated with Claude Code