Skip to content

QTPS update + Blog capability improvement - #43

Merged
dewabisma merged 4 commits into
mainfrom
beast/qtps-update
Sep 5, 2026
Merged

QTPS update + Blog capability improvement#43
dewabisma merged 4 commits into
mainfrom
beast/qtps-update

Conversation

@dewabisma

Copy link
Copy Markdown
Collaborator
  • qtps 430
  • can have draft article
  • can set meta title separate from h1
  • can have adaptive table

@dewabisma
dewabisma requested a review from n13 September 5, 2026 07:58

@n13 n13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewer model: GPT Sol

APPROVE — No blocking findings at 73f04f44ad2ab4c4d2c094aed9f424121188644c.

The publication control is enforced centrally across every blog collection read: production emits only slugs whose en-US source is explicitly published, development retains drafts with a visible preview label, and the 324-file migration adds only published: true. metaTitle affects document/social metadata while the visible H1 and listings retain title. The adaptive-table transform is scoped to blog content, preserves rich cell content, rejects unsupported table shapes during the build, and leaves whitepapers untouched. The QTPS constant also agrees with the current v0.4.0 whitepaper's approximately 430-QTPS figure.

Validation:

  • bun test: 22 passed, 0 failed.
  • Production astro build with SITE_BASE_URL=http://localhost:3000: passed; 1,398 pages generated.
  • Astro Markdown processor integration probe: blog table wrapper/labels present; whitepaper table unchanged.
  • git diff --check: passed.

Non-blocking validation caveats: GitHub Actions run 33954008098 ended in startup_failure before scheduling any jobs. Locally, the repository-wide Prettier check reports 18 files: 16 were already unformatted at the base revision, while the two new files src/utils/blog-posts.ts and src/utils/blog-titles.test.ts add line-wrap-only differences. The SEO check also remains red on 17 pre-existing launch metadata lengths. astro check remains red on pre-existing diagnostics and missing bun:test typings (including the new tests), but the configured build and runtime tests pass.

@dewabisma
dewabisma merged commit 3be140a into main Sep 5, 2026
@dewabisma
dewabisma deleted the beast/qtps-update branch September 5, 2026 08:18
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.

2 participants