Skip to content

docs: record the wake-up adapter and require roadmap updates - #15

Merged
cardmagic merged 2 commits into
mainfrom
agent/roadmap-wake-up
Aug 10, 2026
Merged

docs: record the wake-up adapter and require roadmap updates#15
cardmagic merged 2 commits into
mainfrom
agent/roadmap-wake-up

Conversation

@cardmagic

@cardmagic cardmagic commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Milestone 2 shipped in #14 without the roadmap being updated, so it still claimed PostgreSQL LISTEN/NOTIFY was not implemented and described the cross-process polling delay as the largest unaddressed latency term.

This is the second time a release landed without the roadmap following, so this PR fixes both the stale content and the process that let it go stale.

Roadmap

  • The adapter as implemented, with its measured 103.7 ms to 2.9 ms p50 improvement
  • Why it is opt-in rather than automatic: a connection per waiting thread outside the pool, and LISTEN not surviving a transaction-pooling proxy
  • That MySQL has no notification primitive and keeps polling
  • Milestone 2 reduced to the remaining Redis adapter

AGENTS.md

Requires docs/roadmap.md to be updated in the same pull request whenever a change alters what the project claims about itself: completing or advancing a milestone, moving an entry between implemented and partially implemented, or discovering a limitation worth recording such as a capability that turns out to be opt-in, adapter-specific, or measured rather than assumed.

Two points made explicit, because both were the actual failure modes here: prefer correcting a stale entry over only appending a new one, since an inaccurate claim misleads more than a missing one; and state what was measured and what was not, so a reader can tell evidence from intent. Defect fixes, tooling, and prose changes are exempt.

Documentation only. No version bump.

bundle exec rake   # 301 runs, 0 failures

Milestone 2 shipped without the roadmap being updated, so it still described PostgreSQL notifications as unimplemented and the cross-process polling delay as unaddressed. Record the adapter, its measured latency, why it is opt-in rather than automatic, and that MySQL has no notification primitive. Milestone 2 now covers only the remaining Redis adapter.
@greptile-apps

greptile-apps Bot commented Aug 10, 2026

Copy link
Copy Markdown

Greptile Summary

The roadmap now records the shipped opt-in PostgreSQL LISTEN/NOTIFY wake-up adapter and its measured latency improvement.

  • Documents the adapter’s connection cost and transaction-pooling limitation.
  • Clarifies that MySQL continues polling.
  • Narrows Milestone 2 to the remaining optional Redis adapter.

Confidence Score: 5/5

The documentation-only PR appears safe to merge.

The updated roadmap claims are consistent with the adapter implementation, integration coverage, benchmark documentation, and current backend behavior.

Important Files Changed

Filename Overview
docs/roadmap.md Accurately updates implemented capabilities, operational constraints, benchmark results, and remaining wake-up work.

Reviews (1): Last reviewed commit: "docs: record the PostgreSQL wake-up adap..." | Re-trigger Greptile

Two releases shipped milestones without the roadmap following, leaving it claiming that implemented capabilities were missing. Make the update part of the pull request that changes what the project claims about itself, and say plainly that correcting a stale entry matters more than appending a new one.
@cardmagic cardmagic changed the title docs: record the PostgreSQL wake-up adapter docs: record the wake-up adapter and require roadmap updates Aug 10, 2026
@cardmagic
cardmagic merged commit f632c4a into main Aug 10, 2026
12 checks passed
@cardmagic
cardmagic deleted the agent/roadmap-wake-up branch August 10, 2026 13:52
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