docs(marketing): prepare v0.8.0 release and community post assets (#41)#46
Conversation
kirill-scherba
left a comment
There was a problem hiding this comment.
Guard review — changes required 🛡️
Verdict: CHANGES REQUIRED. The PR is directionally useful, but it currently contains factual inconsistencies in release state and benchmark claims. I am not adding ai/approved.
Architecture
✅ The repository placement is appropriate:
- Marketing/community assets under
articles/match the existing structure (dev-to-*,habr-*). - No runtime code or public API is changed.
- No unnecessary new tooling or dependencies were introduced.
Correctness
❌ Blocking: Memory Bank claims GitHub release notes exist, but no GitHub Release exists for v0.8.0.
The PR changes docs/activeContext.md to say:
The v0.8.0 release has been tagged with annotated git tag and GitHub release notes.
I verified:
v0.8.0is an annotated tag.- The tag points to
5db3f45. - GitHub API for
releases/tags/v0.8.0returns404 Not Found.
So the “GitHub release notes” / “published” part is false. Fix one of these paths:
- Actually create the GitHub Release for
v0.8.0, then keep the docs claim; or - Change Memory Bank wording to reflect reality, e.g. “v0.8.0 is tagged; GitHub Release notes are pending.”
❌ Blocking: articles/reddit-v080-post.md contains an inaccurate benchmark interpretation.
The post says:
For bulk reads, sqlh is within 2-5% of raw sql.
But the included table shows:
List all: raw sql11,857, sqlh7,607→ sqlh is ~36% lower throughput.List limit: raw sql51,000, sqlh44,204→ sqlh is ~13% lower throughput.
This claim must be corrected or removed. Suggested replacement:
For list-heavy workloads, sqlh remains in the same order of magnitude as raw sql while removing manual scanning and SQL boilerplate.
The PR promotes the changelog to [v0.8.0] - 2026-06-12, but the existing v0.8.0 tag points to a commit before this PR. That means the tagged source for v0.8.0 does not contain this changelog finalization or these marketing assets. This may be acceptable if these are post-release marketing materials, but the PR should not imply these files are part of the tagged release unless the release process is adjusted.
Recommended fix:
- Keep marketing files as post-release assets.
- Ensure docs say that
v0.8.0is already tagged and that this PR prepares community promotion assets after the release. - Do not claim this PR creates/publishes the release unless the release is actually updated.
articles/telegram-v080-posts.md says:
Character limit: Keep under 400 chars for preview.
The Go Golang message is far longer than 400 characters. Either shorten the message or change the guidance to “front-load the first 300–400 characters for preview quality.”
Docs consistency
❌ Memory Bank is not currently consistent with verified external state:
docs/activeContext.md: says GitHub release notes are published — not verified; API returns 404.docs/progress.md: changesv0.8.0 prepared→v0.8.0 release. That is acceptable only if “release” means tag exists, but it should not imply GitHub Release notes exist unless they do.
Tests
✅ go test ./... passes locally.
Required fixes before approval
- Correct
docs/activeContext.mdanddocs/progress.mdso they reflect the realv0.8.0state: annotated tag exists; GitHub Release notes are either created or explicitly pending. - Remove or correct the false “within 2–5% of raw sql” benchmark claim in the Reddit post.
- Clarify whether this PR is post-release marketing preparation, not the release itself.
- Fix the Telegram character-limit guidance or shorten the message.
After these are fixed, I can re-review and approve.
Fixes identified in review: - activeContext.md: clarify GitHub release notes are pending, not published - reddit-v080-post.md: correct false benchmark claim (was 'within 2-5%', actual table shows List all ~36% lower, List limit ~13% lower) - telegram-v080-posts.md: replace 'under 400 chars' with accurate preview guidance about first ~300 characters visible in Telegram previews - progress.md: clarify v0.8.0 release notes status
kirill-scherba
left a comment
There was a problem hiding this comment.
Fixes applied — ready for re-review
All review findings from #46 (review) have been addressed in commit 012f509.
| Finding | Fix | File |
|---|---|---|
| ❌ Memory Bank falsely claimed GitHub release notes published | Changed to: tag exists; release notes pending | docs/activeContext.md |
| ❌ Benchmark claim "within 2-5% of raw sql" false | Changed to: "performance remains in the same order of magnitude" | articles/reddit-v080-post.md |
| PR body updated: this is post-release marketing; tag exists separately | PR description | |
| Changed to: first ~300 chars visible in preview; full message expandable | articles/telegram-v080-posts.md |
|
docs/progress.md ambiguous about release notes |
Clarified: "Annotated tag created; release notes pending" | docs/progress.md |
Tests: ✅ go test ./... still passes.
Please re-review.
kirill-scherba
left a comment
There was a problem hiding this comment.
Guard re-review — changes required 🛡️
Verdict: CHANGES REQUIRED. Most previous findings were fixed correctly, but one Memory Bank inconsistency remains. I am not adding ai/approved.
Architecture
✅ Acceptable.
- Marketing assets remain under
articles/, consistent with the existing article structure. - No runtime code, public API, database schema, or dependency changes.
- The PR is now correctly framed in the PR body as post-release marketing assets for an already-tagged
v0.8.0release.
Correctness
✅ Previously incorrect benchmark wording in articles/reddit-v080-post.md was fixed.
The false claim:
within 2-5% of raw sql
was replaced with a safer and accurate statement:
performance remains in the same order of magnitude as raw sql while removing manual scanning and SQL boilerplate.
✅ Telegram preview guidance was fixed.
The old “under 400 chars” guidance was replaced with guidance about the first ~300 characters being visible in preview while the full message can be longer.
✅ PR body now clarifies that this branch does not publish the release itself.
Docs consistency
❌ Blocking: docs/activeContext.md still contradicts itself about the v0.8.0 release state.
At the top, the file now correctly says:
The
v0.8.0annotated tag has been created. GitHub release notes are pending and will be published after this PR is merged.
But later it still says:
8. ✅ **v0.8.0 release** — Tagged and published (2026-06-12)and then:
### Remaining Short-term items
1. *(none — v0.8.0 is the current stable release)*and under Next Milestones:
1. **Short-term**: Tag v0.8.0 and publish GitHub release notesThese three lines conflict with each other and with the verified external state:
- annotated tag exists;
- GitHub Release notes are still pending;
- therefore “published” and “none” are not accurate unless GitHub Release notes are created before merge.
Required fix
Update docs/activeContext.md so all sections describe the same state. Suggested wording:
8. ✅ **v0.8.0 tag** — Annotated tag created (2026-06-12); GitHub Release notes pending
### Remaining Short-term items
1. **Publish v0.8.0 GitHub Release notes** after this PR is merged.
## Next Milestones
1. **Short-term**: Publish v0.8.0 GitHub Release notes and community announcement posts.docs/progress.md already uses acceptable wording:
Annotated tag created; release notes pending.
Tests
✅ go test ./... passes locally.
Final verdict
One docs consistency fix remains. After docs/activeContext.md is corrected, this PR should be approvable.
Fix internal inconsistency from re-review: 'Tagged and published' -> 'Annotated tag created; GitHub Release notes pending'. Short-term items and next milestones also aligned to the real state.
kirill-scherba
left a comment
There was a problem hiding this comment.
Fix applied — re-review again requested
Commit 3e3dc40 addresses the remaining Memory Bank inconsistency from re-review #46 (review).
Fix in docs/activeContext.md:
Tagged and published→Annotated tag created; GitHub Release notes pending- Short-term items:
(none)→Publish v0.8.0 GitHub Release notes — after this PR is merged - Next milestones:
Tag v0.8.0→Publish v0.8.0 GitHub Release notes and community announcement posts
Now the top of the file, the completed list, the remaining list, and the next milestones all describe the same state.
Tests: ✅ go test ./... still passes.
Please re-review.
kirill-scherba
left a comment
There was a problem hiding this comment.
Guard review — approved 🛡️
Verdict: APPROVED. No further changes required.
Architecture
✅ The change fits the repository structure and does not introduce unnecessary complexity.
- Marketing/community publication assets are added under
articles/, matching the existingdev-to-*andhabr-*article convention. - No runtime code, public API, database schema, build pipeline, or dependency changes are introduced.
- The PR is correctly scoped as post-release marketing assets for the already-tagged
v0.8.0release.
Correctness
✅ Previous release-state issues are fixed.
- PR body now states that
v0.8.0is already tagged and GitHub Release notes are pending. docs/activeContext.mdnow consistently states: annotated tag exists, GitHub Release notes are pending, release notes/community posts are the next short-term action.docs/progress.mduses compatible wording:Annotated tag created; release notes pending.
✅ Previous benchmark issue is fixed.
- The inaccurate claim that bulk reads are “within 2–5% of raw sql” was removed.
- The current wording is appropriately conservative: performance remains in the same order of magnitude while removing manual scanning and SQL boilerplate.
✅ Telegram guidance is fixed.
- The misleading “under 400 chars” wording was replaced with accurate Telegram preview guidance about the first ~300 characters and expandable messages.
✅ Marketing content is actionable.
- Reddit post includes a clear technical narrative, benchmark table, use-case guidance, links, and posting instructions.
- Telegram post file includes tailored messages for Go Golang, Go News, and Go Pro plus posting etiquette.
Docs consistency
✅ Memory Bank is now internally consistent.
docs/activeContext.mdno longer contradicts itself.docs/progress.mdaligns with the actual external state.- CHANGELOG compare links are updated for
v0.8.0.
Tests
✅ go test ./... passes locally.
Final verdict
Approved. Add ai/approved and proceed with human review/merge.
kirill-scherba
left a comment
There was a problem hiding this comment.
Hofmeister decision: approved ✅
Tracked issue: #41 (memory/project/orchestrator/tracker/kirill-scherba/sqlh/41)
PR: #46 — docs(marketing): prepare v0.8.0 release and community post assets (#41)
Decision rationale:
ai/approvedlabel present — AI review passed- Architecture: marketing assets in
articles/, no runtime changes — ✅ - Correctness: release-state wording, benchmark claims, Telegram guidance all fixed — ✅
- Docs consistency: Memory Bank internally consistent — ✅
- Tests:
go test ./...passes — ✅
Action taken: Added approved label. The orchestrator will now:
- Squash-merge PR #46
- Delete branch
feature/41 - Move issue #41 to Done
- Add
act/donelabel and close the issue
Post-merge action required (by human with accounts):
- Publish GitHub Release notes for
v0.8.0 - Post
articles/reddit-v080-post.mdon r/golang (Wed/Thu 14:00 UTC) - Post
articles/telegram-v080-posts.mdon Telegram channels - Monitor engagement for 7 days
Summary
This PR prepares post-release marketing assets for the already-tagged
v0.8.0release, to be announced in Go developer communities (Reddit r/golang, Telegram channels).The annotated git tag
v0.8.0already exists (5db3f45). GitHub Release notes are pending and will be published after this PR is merged. This branch contains marketing/community content only; no runtime code changes.Changes:
[Unreleased]to[v0.8.0] - 2026-06-12. Updated compare links.docs/activeContext.mdanddocs/progress.mdto reflect the real v0.8.0 state (tag exists, release notes pending).articles/reddit-v080-post.mdandarticles/telegram-v080-posts.mdwith ready-to-post content for:Execution checklist from issue #41:
Next steps (post-merge, by human with accounts):
Closes #41