* feat(cost): persist run_id on generated posts + back-populate for v0.8.1 reaper
The orphan-research-row reaper (upcoming commit) needs a cheap way to
answer "which posts belong to this run_id?" when amortizing a stranded
`llm_research` row back to its siblings. Relying on the generation_log
post_id column is circular: if the pipeline died before amortize_research
ran, we'd trust the same table we're trying to repair.
- `Publisher::build_meta()` now accepts `?string $run_id` and writes
`_prautoblogger_run_id => $run_id` when non-empty. Null/empty preserves
existing null-safe legacy behaviour.
- `Publisher::create_post()` forwards the $run_id it already receives
into build_meta — one-line plumbing, no new parameters at call sites.
- `Activator::backpopulate_run_id_meta_v081()` runs once per site,
gated by `prautoblogger_migrated_run_id_meta_v081` option. Walks
`wp_prautoblogger_generation_log`, pairs (post_id, min(run_id)),
writes `_prautoblogger_run_id` on any post that doesn't already have
it. Uses `metadata_exists()` so it's a no-op on re-run.
- Activator also schedules the new daily `prautoblogger_reap_orphan_research_rows`
cron at 03:15 — 15 min after the primary generation cron — so live
pipelines have time to finish amortizing cleanly before the reaper scans.
- Deactivator unschedules the new hook alongside the existing cron hooks.
No user-visible behaviour change yet. The reaper handler ships in the
next commit; this one just ensures it will have the data + schedule it
needs when it lands.
Part of the v0.8.1 bundle (thread research-cost-reaper).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* feat(cost): daily reaper for orphan llm_research rows
Implements the orphan-research-row reaper per thread research-cost-reaper
seq 1. When a pipeline dies between post creation and
`Post_Assembler::amortize_research_costs()` (Hostinger exec kill, OOM,
LiteSpeed cut, fatal), the shared research row stays in
`wp_prautoblogger_generation_log` with `post_id = NULL` and sibling
articles' cost popovers understate their true spend.
New class `PRAutoBlogger_Research_Reaper` owns:
- `on_cron()` — WP-Cron entrypoint hooked to the daily
`prautoblogger_reap_orphan_research_rows` event (already scheduled
by the Activator in the previous commit at 03:15).
- `reap()` — returns a `{reaped, deleted, skipped}` summary. For each
orphan `llm_research` row older than 1 hour (grace window for live
pipelines):
- `_prautoblogger_run_id` post_meta primary lookup.
- Falls back to distinct post_ids in gen_log with the same run_id.
- If ≥1 post found → delegate to the existing idempotent
`Post_Assembler::amortize_research_costs()`. That method inserts
the per-article rows and deletes the original orphan in one pass,
so the reaper is idempotent on re-run.
- If 0 posts found AND orphan is >7 days old → delete outright as
sunk cost with nothing to attribute to.
- Else → skip; try again the next day.
- Never throws. DB errors caught and logged WARNING.
New WP-CLI command `wp prautoblogger reap-research` registered in the
main loader when `WP_CLI` is defined — manual ops trigger that prints
the same summary.
Departure from the CTO's spec: the reap logic lives on
`Research_Reaper` instead of `Post_Assembler`. Placing it on
Post_Assembler pushed that file to 391 lines, over the 300-line cap.
Moving it to the thin class colocates the attribution logic with the
cron/CLI plumbing that triggers it. `amortize_research_costs()` stays
on Post_Assembler — the reaper delegates to it unchanged. Flagged for
CTO review.
Part of the v0.8.1 bundle (thread research-cost-reaper).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* test(cost): ResearchReaperTest — five cases per spec
Covers the five scenarios from convo thread research-cost-reaper seq 1:
- test_reaps_orphan_with_matching_run_id_posts — meta lookup returns
two post IDs; reaper delegates to amortize_research_costs which
inserts two rows and deletes the orphan.
- test_skips_orphan_within_grace_period — the SELECT's
`created_at < now - 1h` clause filters fresh orphans at the DB; the
reaper's stats stay zero and no side effects fire.
- test_deletes_orphan_with_no_articles_after_7_days — orphan is 8 days
old, zero siblings via meta or gen_log; reaper deletes outright.
- test_falls_back_to_gen_log_when_post_meta_missing — meta returns
empty (legacy pre-v0.8.1 post); gen_log has two distinct post_ids;
amortize still runs.
- test_idempotent_on_repeat_run — second pass on a cleaned state
returns 0/0/0 with no inserts or deletes.
All DB interaction mocked via $wpdb stub — no real DB round-trips.
Fixtures are driven by property mutations in the test methods so
PHPUnit's "one stub per method" restriction on mocks doesn't bite.
Part of the v0.8.1 bundle.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* chore(v0.8.1): version bump + CHANGELOG + ARCHITECTURE.md for reaper
Bump plugin header and PRAUTOBLOGGER_VERSION 0.8.0 → 0.8.1. Patch
bump — the reaper is internal infrastructure; no user-visible UX
change.
CHANGELOG:
- Fixed: orphan `llm_research` rows now reaped daily.
- Added: WP-CLI `wp prautoblogger reap-research`; `_prautoblogger_run_id`
post_meta is now written by Publisher on every post.
ARCHITECTURE.md:
- Decision #13 (Run-ID based log linking) cross-refs the new post_meta.
- New Decision #19 documents the reaper's scan/amortize/delete policy,
the 1-hour grace window, the 7-day stale cutoff, and the one-shot
activator back-populate migration.
DB schema unchanged — no PRAUTOBLOGGER_DB_VERSION bump.
Final commit of the v0.8.1 bundle.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* test(cost): widen $wpdb mock so ResearchReaperTest actually runs
BaseTestCase::create_mock_wpdb() only adds methods
[prepare, get_var, get_results, insert, query, get_row, update] — the
reaper and amortize_research_costs also call `delete` and `get_col`,
which the default mock can't stub. CI on PR #100 fired all five new
ResearchReaperTest cases as errors at the first `->method('delete')`
call during setUp.
Fix: build a bespoke $wpdb mock in this file's setUp with the full
superset of methods it needs. Self-contained; no change to
BaseTestCase so existing tests aren't affected.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* test(cost): filter gen_log inserts from event_log inserts in reaper tests
CI on ec1efee showed the two reap-and-amortize tests failing because
`assertCount(2, $this->inserted_rows)` was seeing 4 rows, not 2. Root
cause: the Logger singleton writes INFO lines via `$wpdb->insert` into
the event_log table — the reaper logs once ("Reaped...") and amortize
logs once ("Amortized LLM research cost...") before the two per-post
gen_log inserts, so the capture ended up with 4 total rows.
Add a `gen_log_inserts()` helper that filters the capture down to rows
whose `__table` contains "generation_log" — the assertions on
attribution count should only measure the actual cost inserts, not
incidental log writes. Two tests updated; logic unchanged.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(admin): correct misleading GA4 Property ID description
The GA4 client at includes/core/class-ga4-client.php:85 already wraps the
property_id option with "properties/" in the outbound URL path. The admin
description told users to enter "properties/XXXXXXXXX" — which would have
produced a double-prefixed URL (properties/properties/XXXXXXXXX) and a
guaranteed 404 from the Data API. Description now asks for digits only,
matching actual code behavior.
No code logic change — UX copy + CHANGELOG only. Flagged by CEO while
inspecting the Analytics tab; folded into the in-flight v0.8.1 PR per
CTO call rather than cutting a separate patch.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---------
Co-authored-by: peptiderepo <peptiderepo@users.noreply.github.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
159 lines
6.4 KiB
PHP
Executable File
159 lines
6.4 KiB
PHP
Executable File
<?php
|
|
declare(strict_types=1);
|
|
|
|
/**
|
|
* Creates WordPress posts from generated and editor-approved content.
|
|
*
|
|
* Handles both publishing (editor-approved) and saving as draft (editor-rejected).
|
|
* Stores all generation metadata as post_meta for transparency and auditability.
|
|
*
|
|
* Triggered by: PRAutoBlogger_Pipeline_Runner (step 6).
|
|
* Dependencies: PRAutoBlogger_Post_Assembler (taxonomy, images, sanitization),
|
|
* WordPress wp_insert_post(), post meta API.
|
|
*
|
|
* @see core/class-post-assembler.php — Post-creation helpers (taxonomy, images, logs).
|
|
* @see core/class-peptide-linker.php — Injects peptide database hyperlinks into content.
|
|
* @see core/class-chief-editor.php — Produces the editorial review we consume.
|
|
* @see core/class-content-generator.php — Produces the content we publish.
|
|
* @see ARCHITECTURE.md — Data flow step 6.
|
|
*/
|
|
class PRAutoBlogger_Publisher {
|
|
|
|
/**
|
|
* Publish an editor-approved article.
|
|
*
|
|
* @param string $content The final HTML content.
|
|
* @param PRAutoBlogger_Article_Idea $idea The original article idea.
|
|
* @param PRAutoBlogger_Editorial_Review $review The editor's review.
|
|
* @param string|null $run_id Pipeline run ID for log linking.
|
|
* @return int The created post ID.
|
|
* @throws \RuntimeException If post creation fails.
|
|
*/
|
|
public function publish(
|
|
string $content,
|
|
PRAutoBlogger_Article_Idea $idea,
|
|
PRAutoBlogger_Editorial_Review $review,
|
|
?string $run_id = null
|
|
): int {
|
|
return $this->create_post( $content, $idea, $review, 'publish', $run_id );
|
|
}
|
|
|
|
/**
|
|
* Save an editor-rejected article as a draft for human review.
|
|
*
|
|
* @param string $content The generated HTML content (pre-revision).
|
|
* @param PRAutoBlogger_Article_Idea $idea The original article idea.
|
|
* @param PRAutoBlogger_Editorial_Review $review The editor's review with rejection notes.
|
|
* @param string|null $run_id Pipeline run ID for log linking.
|
|
* @return int The created post ID.
|
|
* @throws \RuntimeException If post creation fails.
|
|
*/
|
|
public function save_as_draft(
|
|
string $content,
|
|
PRAutoBlogger_Article_Idea $idea,
|
|
PRAutoBlogger_Editorial_Review $review,
|
|
?string $run_id = null
|
|
): int {
|
|
return $this->create_post( $content, $idea, $review, 'draft', $run_id );
|
|
}
|
|
|
|
/**
|
|
* Create a WordPress post with generation metadata.
|
|
*
|
|
* @param string $content HTML content.
|
|
* @param PRAutoBlogger_Article_Idea $idea Article idea.
|
|
* @param PRAutoBlogger_Editorial_Review $review Editorial review.
|
|
* @param string $post_status 'publish' or 'draft'.
|
|
* @param string|null $run_id Pipeline run ID for log linking.
|
|
* @return int Post ID.
|
|
* @throws \RuntimeException If wp_insert_post fails.
|
|
*/
|
|
private function create_post(
|
|
string $content,
|
|
PRAutoBlogger_Article_Idea $idea,
|
|
PRAutoBlogger_Editorial_Review $review,
|
|
string $post_status,
|
|
?string $run_id = null
|
|
): int {
|
|
// Clean LLM artifacts, then inject peptide hyperlinks deterministically
|
|
// before the content enters WordPress. Peptide linker no-ops gracefully
|
|
// if PR Core is not active.
|
|
$clean_content = PRAutoBlogger_Post_Assembler::sanitize_llm_content( $content );
|
|
$linked_content = PRAutoBlogger_Peptide_Linker::inject_links( $clean_content );
|
|
|
|
$post_data = [
|
|
'post_title' => sanitize_text_field( $idea->get_suggested_title() ),
|
|
'post_content' => wp_kses_post( $linked_content ),
|
|
'post_status' => $post_status,
|
|
'post_type' => 'post',
|
|
'post_author' => PRAutoBlogger_Post_Assembler::get_default_author_id(),
|
|
'meta_input' => $this->build_meta( $idea, $review, $run_id ),
|
|
];
|
|
|
|
/** @see class-prautoblogger.php — listeners registered in main loader. */
|
|
$post_data = apply_filters( 'prautoblogger_filter_post_data', $post_data, $idea, $review );
|
|
|
|
$post_id = wp_insert_post( $post_data, true );
|
|
if ( is_wp_error( $post_id ) ) {
|
|
throw new \RuntimeException(
|
|
sprintf( __( 'Failed to create post: %s', 'prautoblogger' ), $post_id->get_error_message() )
|
|
);
|
|
}
|
|
|
|
PRAutoBlogger_Post_Assembler::assign_taxonomy_terms( $post_id, $idea );
|
|
PRAutoBlogger_Post_Assembler::link_generation_logs( $post_id, $run_id );
|
|
|
|
if ( 'publish' === $post_status ) {
|
|
PRAutoBlogger_Post_Assembler::attach_generated_images( $post_id, $idea, $post_data );
|
|
}
|
|
|
|
PRAutoBlogger_Logger::instance()->info(
|
|
sprintf( 'Post created: ID=%d, status=%s, title="%s"', $post_id, $post_status, $idea->get_suggested_title() ),
|
|
'publisher'
|
|
);
|
|
|
|
do_action( 'prautoblogger_post_created', $post_id, $post_status, $idea, $review );
|
|
|
|
return $post_id;
|
|
}
|
|
|
|
/**
|
|
* Build post_meta array for generation metadata.
|
|
*
|
|
* The `_prautoblogger_run_id` key (added v0.8.1) lets the orphan-research
|
|
* reaper attribute an orphan `llm_research` row back to its sibling
|
|
* articles without re-walking the gen_log table. See
|
|
* core/class-research-reaper.php.
|
|
*
|
|
* @param PRAutoBlogger_Article_Idea $idea
|
|
* @param PRAutoBlogger_Editorial_Review $review
|
|
* @param string|null $run_id Pipeline run UUID, or null in legacy paths.
|
|
* @return array<string, mixed>
|
|
*/
|
|
private function build_meta(
|
|
PRAutoBlogger_Article_Idea $idea,
|
|
PRAutoBlogger_Editorial_Review $review,
|
|
?string $run_id = null
|
|
): array {
|
|
$meta = [
|
|
'_prautoblogger_generated' => '1',
|
|
'_prautoblogger_analysis_id' => $idea->get_analysis_id(),
|
|
'_prautoblogger_source_ids' => wp_json_encode( $idea->get_source_ids() ),
|
|
'_prautoblogger_model_used' => get_option( 'prautoblogger_writing_model', PRAUTOBLOGGER_DEFAULT_WRITING_MODEL ),
|
|
'_prautoblogger_pipeline_mode' => get_option( 'prautoblogger_writing_pipeline', 'multi_step' ),
|
|
'_prautoblogger_editor_verdict' => $review->get_verdict(),
|
|
'_prautoblogger_editor_notes' => $review->get_notes(),
|
|
'_prautoblogger_quality_score' => $review->get_quality_score(),
|
|
'_prautoblogger_seo_score' => $review->get_seo_score(),
|
|
'_prautoblogger_generated_at' => gmdate( 'c' ),
|
|
'_prautoblogger_topic' => $idea->get_topic(),
|
|
'_prautoblogger_article_type' => $idea->get_article_type(),
|
|
'_prautoblogger_target_keywords' => wp_json_encode( $idea->get_target_keywords() ),
|
|
];
|
|
if ( null !== $run_id && '' !== $run_id ) {
|
|
$meta['_prautoblogger_run_id'] = $run_id;
|
|
}
|
|
return $meta;
|
|
}
|
|
}
|