Skip to content

[visuals] Add reusable BTST product-proof asset kit - #266

Merged
olliethedev merged 1 commit into
mainfrom
issue/259-product-proof
Sep 1, 2026
Merged

[visuals] Add reusable BTST product-proof asset kit#266
olliethedev merged 1 commit into
mainfrom
issue/259-product-proof

Conversation

@olliethedev

Copy link
Copy Markdown
Collaborator

Summary

  • add a deterministic Playwright/Sharp capture workflow backed by the generated Next.js app and a coherent, non-sensitive BTST fixture
  • add eight reusable static proof assets: README hero, ownership diagram, Blog, Form Builder, UI Builder, OpenAPI, AI Chat, and code-to-result
  • add a manifest/checker for format, dimensions, byte budgets, alt text, captions, editable source, and stale/personal copy
  • wire the new assets into the README and relevant docs pages
  • remove replaced assets, including the 14.6 MB UI Builder GIF, stale Chat and architecture images, and superseded Blog/Form/OpenAPI screenshots; all deletions remain recoverable through Git history

Validation

  • Node 22 / pnpm 10.17.1
  • two identical capture runs produced matching SHA-256 hashes for all eight assets
  • asset checker: 8 assets, 224,951 bytes total under the 1.8 MB limit
  • product-proof checker tests: 4 passing
  • frozen lockfile install
  • repository lint
  • package typecheck
  • production docs build
  • GitHub Markdown API render
  • agent-browser production verification at 1440x900 and 390x844 in light and dark themes, with complete images and no console errors

Closes #259

@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated
better-stack-docs Ignored Ignored Sep 1, 2026 7:21pm UTC
better-stack-playground Ignored Ignored Sep 1, 2026 7:21pm UTC

Request Review

@olliethedev
olliethedev force-pushed the issue/259-product-proof branch 3 times, most recently from 30ee86c to 43073ff Compare September 1, 2026 19:20
@olliethedev
olliethedev force-pushed the issue/259-product-proof branch from 43073ff to 8dc311c Compare September 1, 2026 19:21
@olliethedev
olliethedev marked this pull request as ready for review September 1, 2026 19:22
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-01T19:25:34.357343Z 8dc311c Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8dc311ca8a

ℹ️ 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".

await codeToResult(blog, blogRegistrationProof);
} finally {
try {
await cleanupMedia(context.request);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Remove all seeded records in the finalizer

When this capture runs against a generated app that is subsequently reused, the finally block deletes only the Media fixture, leaving the three Blog posts, Comment, and UI Builder page in the shared adapter. A failed run also leaves whichever of those records were created before the failure. This contradicts the documented before-and-after cleanup contract and can change later E2E results or leave dogfood content visible; track and delete all seeded fixture records here alongside the Media cleanup.

Useful? React with 👍 / 👎.

@olliethedev
olliethedev merged commit 222c585 into main Sep 1, 2026
13 of 17 checks passed
@olliethedev
olliethedev deleted the issue/259-product-proof branch September 1, 2026 19:35
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.

[visuals] Create the reusable BTST product-proof asset kit

1 participant