← All changes | includes/admin/class-metabox-manager.php +398 -37 2.1.1 → 2.10.0 View file →

		@@ -18,8 +18,9 @@
18	18	use ThinkRank\Core\Settings;
19	19	use ThinkRank\Core\Database;
20	20	use ThinkRank\Core\Plan_Config;
21	21	use ThinkRank\SEO\Focus_Keywords;
	22	+use ThinkRank\SEO\Object_Redirect;
22	23	use ThinkRank\SEO\Pattern_Resolver;
23	24
24	25	// Prevent direct access
25	26	if (!defined('ABSPATH')) {
		@@ -77,8 +78,9 @@
77	78	public function init(): void {
78	79	add_action('add_meta_boxes', [$this, 'add_meta_boxes']);
79	80	add_action('save_post', [$this, 'save_meta_boxes'], 10, 2);
80	81	add_action('admin_enqueue_scripts', [$this, 'enqueue_metabox_scripts']);
	82	+ add_action('admin_notices', [$this, 'render_redirect_notice']);
81	83	add_action('init', [$this, 'register_meta_fields']);
82	84
83	85	// AJAX handlers for meta box functionality
84	86	add_action('wp_ajax_thinkrank_generate_post_metadata', [$this, 'ajax_generate_post_metadata']);
		@@ -206,8 +208,26 @@
206	208	return current_user_can('edit_posts') || current_user_can('edit_pages');
207	209	}
208	210	]);
209	211
	212	+ register_post_meta('', \ThinkRank\SEO\Content_Visibility::SEARCH_META, [
	213	+ 'show_in_rest' => true,
	214	+ 'single' => true,
	215	+ 'type' => 'integer',
	216	+ 'auth_callback' => function () {
	217	+ return current_user_can('edit_posts') || current_user_can('edit_pages');
	218	+ }
	219	+ ]);
	220	+
	221	+ register_post_meta('', \ThinkRank\SEO\Content_Visibility::ARCHIVE_META, [
	222	+ 'show_in_rest' => true,
	223	+ 'single' => true,
	224	+ 'type' => 'integer',
	225	+ 'auth_callback' => function () {
	226	+ return current_user_can('edit_posts') || current_user_can('edit_pages');
	227	+ }
	228	+ ]);
	229	+
210	230	register_post_meta('', '_thinkrank_primary_category', [
211	231	'show_in_rest' => true,
212	232	'single' => true,
213	233	'type' => 'integer',
		@@ -483,9 +503,22 @@
483	503	<input type="hidden" id="thinkrank_focus_keywords" name="thinkrank_focus_keywords" value="<?php echo esc_attr(wp_json_encode($existing_metadata['focus_keywords'] ?? [])); ?>" />
484	504	<input type="hidden" id="thinkrank_seo_score" name="thinkrank_seo_score" value="<?php echo esc_attr($existing_metadata['seo_score'] ?? '0'); ?>" />
485	505	<input type="hidden" id="thinkrank_generated_at" name="thinkrank_generated_at" value="<?php echo esc_attr($existing_metadata['generated_at'] ?? ''); ?>" />
486	506	<input type="hidden" id="thinkrank_pillar_content" name="thinkrank_pillar_content" value="<?php echo esc_attr($existing_metadata['pillar_content'] ?? ''); ?>" />
	507	+ <input type="hidden" id="thinkrank_exclude_from_search" name="thinkrank_exclude_from_search" value="<?php echo esc_attr((string) ($existing_metadata['exclude_from_search'] ?? '')); ?>" />
	508	+ <input type="hidden" id="thinkrank_exclude_from_archives" name="thinkrank_exclude_from_archives" value="<?php echo esc_attr((string) ($existing_metadata['exclude_from_archives'] ?? '')); ?>" />
487	509	<input type="hidden" id="thinkrank_canonical_url" name="thinkrank_canonical_url" value="<?php echo esc_url($existing_metadata['canonical_url'] ?? ''); ?>" />
	510	+ <?php
	511	+ // The redirect lives in Pro's rules table, not post meta, so nothing
	512	+ // else hands it to the React app. Without these the field loads
	513	+ // empty, and its own hidden input then posts that empty value on the
	514	+ // next save, which Object_Redirect reads as "remove the redirect".
	515	+ // Rendered only when a provider can store it, matching MetaboxApp.
	516	+ if (Object_Redirect::is_supported()) :
	517	+ ?>
	518	+ <input type="hidden" id="thinkrank_redirect_url" name="thinkrank_redirect_url" value="<?php echo esc_attr((string) ($existing_metadata['redirect_url'] ?? '')); ?>" />
	519	+ <input type="hidden" id="thinkrank_redirect_type" name="thinkrank_redirect_type" value="<?php echo esc_attr((string) ($existing_metadata['redirect_type'] ?? Object_Redirect::DEFAULT_TYPE)); ?>" />
	520	+ <?php endif; ?>
488	521	<input type="hidden" id="thinkrank_robots_meta_enabled" name="thinkrank_robots_meta_enabled" value="<?php echo esc_attr((string) ($existing_metadata['robots_meta_enabled'] ?? '0')); ?>" />
489	522	<input type="hidden" id="thinkrank_robots_meta" name="thinkrank_robots_meta" value="<?php echo esc_attr((string) ($existing_metadata['robots_meta'] ?? '')); ?>" />
490	523	<input type="hidden" id="thinkrank_advanced_robots_meta" name="thinkrank_advanced_robots_meta" value="<?php echo esc_attr((string) ($existing_metadata['advanced_robots_meta'] ?? '')); ?>" />
491	524	<input type="hidden" id="thinkrank_og_title" name="thinkrank_og_title" value="<?php echo esc_attr((string) ($existing_metadata['og_title'] ?? '')); ?>" />
		@@ -532,8 +565,16 @@
532	565	// #post form, so they arrive (slashed) in $_POST. Hand them straight to
533	566	// the shared persistence routine.
534	567	// phpcs:ignore WordPress.Security.NonceVerification.Missing -- nonce verified above
535	568	$this->persist_metadata($post_id, wp_unslash($_POST));
	569	+
	570	+ // The redirect is the one field here that can be refused outright. The
	571	+ // response to this request is a redirect back to the editor, so the
	572	+ // reason has to survive one page load to be seen at all.
	573	+ $redirect_error = $this->get_last_redirect_error();
	574	+ if (null !== $redirect_error) {
	575	+ $this->store_redirect_error($redirect_error);
	576	+ }
536	577	}
537	578
538	579	/**
539	580	* Persist metabox fields for a post from a form-field-name => value map,
		@@ -582,8 +623,10 @@
582	623	$fields = [
583	624	'thinkrank_seo_score' => 'absint',
584	625	'thinkrank_generated_at' => 'sanitize_text_field',
585	626	'thinkrank_pillar_content' => 'sanitize_text_field',
	627	+ 'thinkrank_exclude_from_search' => 'sanitize_text_field',
	628	+ 'thinkrank_exclude_from_archives' => 'sanitize_text_field',
586	629	];
587	630
588	631	// Focus keywords: prefer the JSON array field; fall back to the legacy
589	632	// single string. Focus_Keywords::save() normalizes (dedupe, drop empty,
		@@ -623,8 +666,10 @@
623	666	update_post_meta($post_id, '_thinkrank_canonical_url', $canonical_url);
624	667	}
625	668	}
626	669
	670	+ $this->last_redirect_error = $this->save_object_redirect('post', $post_id, $src);
	671	+
627	672	foreach ($fields as $field => $sanitize_callback) {
628	673	if (isset($src[$field])) {
629	674	$value = call_user_func($sanitize_callback, $src[$field]);
630	675	update_post_meta($post_id, "_{$field}", $value);
		@@ -631,8 +676,9 @@
631	676	}
632	677	}
633	678
634	679	$this->save_robots_meta($post_id, $src);
	680	+ $this->save_visibility_meta($post_id, $src);
635	681	$this->save_social_meta($post_id, $src);
636	682
637	683	// Update last modified timestamp
638	684	update_post_meta($post_id, '_thinkrank_last_updated', current_time('mysql'));
		@@ -807,8 +853,43 @@
807	853	}
808	854
809	855
810	856	/**
	857	+ * Save the per-post listing-visibility switches.
	858	+ *
	859	+ * Stored as 1 or deleted rather than 1/0: the excluded set is read with a
	860	+ * `meta_value = '1'` query, so a row holding 0 would be dead weight on every
	861	+ * post anyone ever unticked. Deleting keeps the postmeta table proportional
	862	+ * to the number of posts actually hidden.
	863	+ *
	864	+ * @since 2.7.0
	865	+ *
	866	+ * @param int $post_id Post being saved.
	867	+ * @param array $src Submitted fields.
	868	+ * @return void
	869	+ */
	870	+ private function save_visibility_meta(int $post_id, array $src): void {
	871	+ $fields = [
	872	+ 'thinkrank_exclude_from_search' => \ThinkRank\SEO\Content_Visibility::SEARCH_META,
	873	+ 'thinkrank_exclude_from_archives' => \ThinkRank\SEO\Content_Visibility::ARCHIVE_META,
	874	+ ];
	875	+
	876	+ foreach ($fields as $field => $meta_key) {
	877	+ if (!isset($src[$field])) {
	878	+ continue;
	879	+ }
	880	+
	881	+ if ((bool) $src[$field]) {
	882	+ update_post_meta($post_id, $meta_key, 1);
	883	+ } else {
	884	+ delete_post_meta($post_id, $meta_key);
	885	+ }
	886	+ }
	887	+
	888	+ \ThinkRank\SEO\Content_Visibility::flush();
	889	+ }
	890	+
	891	+ /**
811	892	* Enqueue meta box scripts
812	893	*
813	894	* @param string $hook Current admin page hook
814	895	* @return void
		@@ -940,8 +1021,13 @@
940	1021	'postModified' => $post ? get_the_modified_date('c', $post) : '',
941	1022	'linkSuggestionsEnabled' => $this->is_link_suggestions_enabled($post_type),
942	1023	'postStatus' => get_post_status($post_id),
943	1024	'isPro' => Plan_Config::is_pro(),
	1025	+ // Whether a provider (Pro's Redirections feature) can actually store
	1026	+ // a redirect. False renders the field as an upsell rather than an
	1027	+ // input that accepts text nothing will ever act on.
	1028	+ 'redirectSupported' => Object_Redirect::is_supported(),
	1029	+ 'redirectTypes' => Object_Redirect::TYPES,
944	1030	/**
945	1031	* Filter the editor SEO panel's post-load refresh behaviour.
946	1032	*
947	1033	* The panel re-checks `/metadata/{id}` after load so values written
		@@ -966,16 +1052,13 @@
966	1052	'maxTicks' => 10,
967	1053	],
968	1054	$post_id
969	1055	),
970		- // Whether any AI provider API key is configured — gates the
971		- // "Generate with AI" button in the metabox
972		- 'aiConfigured' => !empty($this->settings->get('openai_api_key', ''))
973		- || !empty($this->settings->get('claude_api_key', ''))
974		- || !empty($this->settings->get('gemini_api_key', ''))
975		- || !empty($this->settings->get('openrouter_api_key', '')),
976		- // Focus keywords plan limits (max_keywords; 0 = unlimited).
977		- 'focusKeywords' => Plan_Config::focus_keywords(),
	1056	+ // Whether the selected AI provider is configured — gates the
	1057	+ // "Generate with AI" button in the metabox. An OpenAI-compatible
	1058	+ // endpoint counts once it has a base URL and a model id, with or
	1059	+ // without a key (#721).
	1060	+ 'aiConfigured' => $this->settings->has_ai_provider_configured(),
978	1061	// Resolved Global/Bulk SEO variable-tag patterns for this post, shown
979	1062	// as placeholder previews when a field is empty (the frontend applies
980	1063	// these same patterns on output). Typing a value overrides them.
981	1064	'patternPreviews' => Pattern_Resolver::previews($post_id),
		@@ -981,8 +1064,12 @@
981	1064	'patternPreviews' => Pattern_Resolver::previews($post_id),
982	1065	// Token => value map (keys without %), for live client-side preview of
983	1066	// a custom pattern typed into a metabox field.
984	1067	'patternVariables' => Pattern_Resolver::variables($post_id),
	1068	+ // The uncapped copy the Content Analysis panel measures. Localized
	1069	+ // here so all six editors get it, rather than each builder adding
	1070	+ // its own (#778). `contentPreview` stays as the AI budget.
	1071	+ 'contentFull' => $post instanceof \WP_Post ? $this->get_analysis_content($post) : '',
985	1072	];
986	1073	}
987	1074
988	1075	/**
		@@ -1091,26 +1178,43 @@
1091	1178	'show_ui' => true,
1092	1179	'_builtin' => false,
1093	1180	]);
1094	1181
	1182	+ // WordPress internals that should never carry an SEO metabox. Fixed,
	1183	+ // so it is built once rather than per post type.
	1184	+ $wp_internal_types = [
	1185	+ 'attachment',
	1186	+ 'revision',
	1187	+ 'nav_menu_item',
	1188	+ 'custom_css',
	1189	+ 'customize_changeset',
	1190	+ 'oembed_cache',
	1191	+ 'user_request',
	1192	+ 'wp_block',
	1193	+ 'wp_template',
	1194	+ 'wp_template_part',
	1195	+ 'wp_global_styles',
	1196	+ 'wp_navigation',
	1197	+ 'acf-field',
	1198	+ 'acf-field-group',
	1199	+ ];
	1200	+
	1201	+ // Builder template CPTs (Bricks, Elementor, Divi, Beaver Builder) are
	1202	+ // layout fragments, not pages with their own SEO. Global SEO already
	1203	+ // refuses them; this list is shared with that policy so the two cannot
	1204	+ // drift apart again (#621).
	1205	+ if (!class_exists('\ThinkRank\SEO\Global_SEO_Post_Types')) {
	1206	+ require_once THINKRANK_PLUGIN_DIR . 'includes/seo/class-global-seo-post-types.php';
	1207	+ }
	1208	+
1095	1209	foreach ($custom_post_types as $post_type) {
1096		- // Skip certain post types that shouldn't have SEO metabox
1097		- $excluded_types = [
1098		- 'attachment',
1099		- 'revision',
1100		- 'nav_menu_item',
1101		- 'custom_css',
1102		- 'customize_changeset',
1103		- 'oembed_cache',
1104		- 'user_request',
1105		- 'wp_block',
1106		- 'wp_template',
1107		- 'wp_template_part',
1108		- 'wp_global_styles',
1109		- 'wp_navigation',
1110		- 'acf-field',
1111		- 'acf-field-group',
1112		- ];
	1210	+ // Resolved per post type, not hoisted: the shared list runs through
	1211	+ // a public filter that receives the post-type object, so an
	1212	+ // integrator can answer differently for different post types.
	1213	+ $excluded_types = array_merge(
	1214	+ $wp_internal_types,
	1215	+ \ThinkRank\SEO\Global_SEO_Post_Types::excluded_post_types(get_post_type_object($post_type))
	1216	+ );
1113	1217
1114	1218	if (!in_array($post_type, $excluded_types, true) && !in_array($post_type, $default_types, true)) {
1115	1219	$default_types[] = $post_type;
1116	1220	}
		@@ -1119,8 +1223,130 @@
1119	1223	return apply_filters('thinkrank_supported_post_types', $default_types);
1120	1224	}
1121	1225
1122	1226	/**
	1227	+ * Transient holding the last redirect error for the current user.
	1228	+ */
	1229	+ private const REDIRECT_ERROR_TRANSIENT = 'thinkrank_redirect_error_';
	1230	+
	1231	+ /**
	1232	+ * Why the redirect field was refused on the most recent persist, if it was.
	1233	+ *
	1234	+ * @var \WP_Error|null
	1235	+ */
	1236	+ private ?\WP_Error $last_redirect_error = null;
	1237	+
	1238	+ /**
	1239	+ * Persist the edit-screen redirect field.
	1240	+ *
	1241	+ * Absent keys are left alone, so a caller that never rendered the field
	1242	+ * (the AJAX save from an editor that submits a subset, an import) cannot
	1243	+ * clear a redirect by omission.
	1244	+ *
	1245	+ * The destination is not post meta — Pro's rules table holds it — so unlike
	1246	+ * every other field here this save can fail for reasons the editor needs to
	1247	+ * hear about: no Pro, plain permalinks, a destination that is the page's own
	1248	+ * URL. Failing silently would be the worst of both, since the field would
	1249	+ * redisplay empty on the next load with no explanation, so the reason is
	1250	+ * stashed for the notice rendered on the next screen.
	1251	+ *
	1252	+ * @param string $object_type 'post' or 'term'.
	1253	+ * @param int $object_id Object ID.
	1254	+ * @param array $src Field name => raw value map.
	1255	+ * @return void
	1256	+ */
	1257	+ private function save_object_redirect(string $object_type, int $object_id, array $src): ?\WP_Error {
	1258	+ if (!array_key_exists('thinkrank_redirect_url', $src)) {
	1259	+ return null;
	1260	+ }
	1261	+
	1262	+ $url = (string) $src['thinkrank_redirect_url'];
	1263	+
	1264	+ // With no provider there is nothing to store and nothing to clear.
	1265	+ // Staying quiet when the field was submitted empty keeps every ordinary
	1266	+ // save on a free site from raising an error about a field the editor
	1267	+ // never touched.
	1268	+ if (!Object_Redirect::is_supported()) {
	1269	+ if ('' !== trim($url)) {
	1270	+ return new \WP_Error(
	1271	+ 'thinkrank_redirect_unsupported',
	1272	+ __('Redirects require ThinkRank Pro with the Redirections feature active.', 'thinkrank')
	1273	+ );
	1274	+ }
	1275	+ return null;
	1276	+ }
	1277	+
	1278	+ $type = array_key_exists('thinkrank_redirect_type', $src)
	1279	+ ? $src['thinkrank_redirect_type']
	1280	+ : Object_Redirect::DEFAULT_TYPE;
	1281	+
	1282	+ $result = Object_Redirect::save($object_type, $object_id, $url, $type);
	1283	+
	1284	+ return is_wp_error($result) ? $result : null;
	1285	+ }
	1286	+
	1287	+ /**
	1288	+ * Why the last persist_metadata() call could not store the redirect.
	1289	+ *
	1290	+ * Every other metabox field either saves or is sanitized into something
	1291	+ * that does; this one can be refused, and each caller reports that
	1292	+ * differently — a notice for the form post, a JSON field for the AJAX save,
	1293	+ * an error message for the MCP ability.
	1294	+ *
	1295	+ * @return \WP_Error|null
	1296	+ */
	1297	+ public function get_last_redirect_error(): ?\WP_Error {
	1298	+ return $this->last_redirect_error;
	1299	+ }
	1300	+
	1301	+ /**
	1302	+ * Remember why a redirect could not be saved, for the next admin screen.
	1303	+ *
	1304	+ * @param \WP_Error $error Failure.
	1305	+ * @return void
	1306	+ */
	1307	+ private function store_redirect_error(\WP_Error $error): void {
	1308	+ $user_id = get_current_user_id();
	1309	+ if ($user_id <= 0) {
	1310	+ return;
	1311	+ }
	1312	+
	1313	+ set_transient(self::REDIRECT_ERROR_TRANSIENT . $user_id, $error->get_error_message(), MINUTE_IN_SECONDS);
	1314	+ }
	1315	+
	1316	+ /**
	1317	+ * Show, once, why the last redirect save failed.
	1318	+ *
	1319	+ * @return void
	1320	+ */
	1321	+ public function render_redirect_notice(): void {
	1322	+ $user_id = get_current_user_id();
	1323	+ if ($user_id <= 0) {
	1324	+ return;
	1325	+ }
	1326	+
	1327	+ $key = self::REDIRECT_ERROR_TRANSIENT . $user_id;
	1328	+ $message = get_transient($key);
	1329	+
	1330	+ if (!is_string($message) || '' === $message) {
	1331	+ return;
	1332	+ }
	1333	+
	1334	+ delete_transient($key);
	1335	+
	1336	+ printf(
	1337	+ '<div class="notice notice-error is-dismissible"><p>%s</p></div>',
	1338	+ esc_html(
	1339	+ sprintf(
	1340	+ /* translators: %s: reason the redirect was not saved. */
	1341	+ __('ThinkRank could not save the redirect: %s', 'thinkrank'),
	1342	+ $message
	1343	+ )
	1344	+ )
	1345	+ );
	1346	+ }
	1347	+
	1348	+ /**
1123	1349	* Get existing post metadata
1124	1350	*
1125	1351	* @param int $post_id Post ID
1126	1352	* @return array Existing metadata
		@@ -1125,8 +1351,11 @@
1125	1351	* @param int $post_id Post ID
1126	1352	* @return array Existing metadata
1127	1353	*/
1128	1354	public function get_post_metadata(int $post_id): array {
	1355	+ // One lookup: get() goes through a filter Pro answers from the database.
	1356	+ $redirect = Object_Redirect::get('post', $post_id);
	1357	+
1129	1358	return [
1130	1359	'title' => get_post_meta($post_id, '_thinkrank_seo_title', true),
1131	1360	'description' => get_post_meta($post_id, '_thinkrank_meta_description', true),
1132	1361	'focus_keyword' => Focus_Keywords::get_primary($post_id),
		@@ -1133,9 +1362,15 @@
1133	1362	'focus_keywords' => Focus_Keywords::get($post_id),
1134	1363	'seo_score' => get_post_meta($post_id, '_thinkrank_seo_score', true),
1135	1364	'generated_at' => get_post_meta($post_id, '_thinkrank_generated_at', true),
1136	1365	'pillar_content' => get_post_meta($post_id, '_thinkrank_pillar_content', true),
	1366	+ 'exclude_from_search' => get_post_meta($post_id, \ThinkRank\SEO\Content_Visibility::SEARCH_META, true),
	1367	+ 'exclude_from_archives' => get_post_meta($post_id, \ThinkRank\SEO\Content_Visibility::ARCHIVE_META, true),
1137	1368	'canonical_url' => get_post_meta($post_id, '_thinkrank_canonical_url', true),
	1369	+ // Not post meta: the rule in Pro's redirections table is the value.
	1370	+ // See ThinkRank\SEO\Object_Redirect.
	1371	+ 'redirect_url' => $redirect['url'],
	1372	+ 'redirect_type' => $redirect['type'],
1138	1373	'robots_meta_enabled' => get_post_meta($post_id, '_thinkrank_robots_meta_enabled', true),
1139	1374	'robots_meta' => get_post_meta($post_id, '_thinkrank_robots_meta', true),
1140	1375	'advanced_robots_meta' => get_post_meta($post_id, '_thinkrank_advanced_robots_meta', true),
1141	1376	'og_title' => get_post_meta($post_id, '_thinkrank_og_title', true),
		@@ -1214,18 +1449,68 @@
1214	1449	return null;
1215	1450	}
1216	1451
1217	1452	/**
1218		- * Get content preview for AI analysis
1219		- *
1220		- * @param \WP_Post $post Post object
1221		- * @return string Content preview
	1453	+ * Characters of page text an AI caller is given.
	1454	+ *
	1455	+ * A prompt budget, not a description of the page — one caller forwards this
	1456	+ * straight into an AI payload, so it stays capped.
	1457	+ *
	1458	+ * @since 2.10.0
	1459	+ * @var int
1222	1460	*/
1223		- public function get_content_preview(\WP_Post $post): string {
1224		- $content = $post->post_title . "\n\n";
	1461	+ private const AI_PREVIEW_LENGTH = 4000;
1225	1462
1226		- if (!empty($post->post_excerpt)) {
1227		- $content .= $post->post_excerpt . "\n\n";
	1463	+ /**
	1464	+ * Safety ceiling for the analysis copy.
	1465	+ *
	1466	+ * High enough that no real article reaches it, low enough that a runaway
	1467	+ * builder tree cannot put a megabyte of text through wp_localize_script
	1468	+ * into every editor page load.
	1469	+ *
	1470	+ * @since 2.10.0
	1471	+ * @var int
	1472	+ */
	1473	+ private const ANALYSIS_MAX_LENGTH = 200000;
	1474	+
	1475	+ /**
	1476	+ * Resolved body text per post, for this request.
	1477	+ *
	1478	+ * Every builder integration localizes get_localized_data() (which carries
	1479	+ * the analysis copy) and then adds get_content_preview() on top, and the
	1480	+ * classic screen renders the preview into the form as well as localizing
	1481	+ * the analysis copy. Each of those resolved the post through
	1482	+ * Builder_Content from scratch, so a builder page walked its whole tree
	1483	+ * twice on every editor load. Both now derive from one resolution.
	1484	+ *
	1485	+ * Keyed on the post's ID and modified time, so a post saved and re-read
	1486	+ * within the same request is resolved again.
	1487	+ *
	1488	+ * @since 2.10.0
	1489	+ * @var array<string,string>
	1490	+ */
	1491	+ private array $resolved_bodies = [];
	1492	+
	1493	+ /**
	1494	+ * The post's body text, resolved through whatever built it.
	1495	+ *
	1496	+ * Body only. The title and excerpt used to be prepended here, which is
	1497	+ * right for an AI prompt but wrong for a measurement: the Analysis panel
	1498	+ * counted them as body copy, so Word Count and Keyword Density in a
	1499	+ * builder included the title and excerpt, while the same panel in the
	1500	+ * block editor (which reads the editor's content) did not. They are added
	1501	+ * back for the AI preview only.
	1502	+ *
	1503	+ * @since 2.10.0
	1504	+ *
	1505	+ * @param \WP_Post $post Post object.
	1506	+ * @return string Plain text, whitespace collapsed, uncapped.
	1507	+ */
	1508	+ private function resolve_post_body(\WP_Post $post): string {
	1509	+ $key = $post->ID . '|' . (string) ($post->post_modified_gmt ?? '');
	1510	+
	1511	+ if (isset($this->resolved_bodies[$key])) {
	1512	+ return $this->resolved_bodies[$key];
1228	1513	}
1229	1514
1230	1515	// Resolve through Builder_Content: a page builder keeps its words
1231	1516	// outside post_content (Oxygen empties it entirely), and the editor
		@@ -1233,18 +1518,80 @@
1233	1518	// browser is just the title and the live panel reports "No content".
1234	1519	if (!class_exists('\\ThinkRank\\SEO\\Builder_Content')) {
1235	1520	require_once THINKRANK_PLUGIN_DIR . 'includes/seo/class-builder-content.php';
1236	1521	}
1237		- $content .= \ThinkRank\SEO\Builder_Content::resolve($post);
1238	1522
1239		- // Clean and limit content
	1523	+ $this->resolved_bodies[$key] = self::collapse_text(\ThinkRank\SEO\Builder_Content::resolve($post));
	1524	+
	1525	+ return $this->resolved_bodies[$key];
	1526	+ }
	1527	+
	1528	+ /**
	1529	+ * Strip tags and collapse whitespace.
	1530	+ *
	1531	+ * @since 2.10.0
	1532	+ *
	1533	+ * @param string $content Markup or text.
	1534	+ * @return string
	1535	+ */
	1536	+ private static function collapse_text(string $content): string {
1240	1537	$content = wp_strip_all_tags($content);
1241		- $content = preg_replace('/\s+/', ' ', $content);
1242	1538
1243		- return trim(substr($content, 0, 4000));
	1539	+ return trim((string) preg_replace('/\s+/', ' ', $content));
1244	1540	}
1245	1541
1246	1542	/**
	1543	+ * The post's text for the editor's Content Analysis panel.
	1544	+ *
	1545	+ * Uncapped, because this is a measurement rather than a budget. In a page
	1546	+ * builder there is no `core/editor` store, no TinyMCE instance and no
	1547	+ * `#content` textarea, so the localized string is the ONLY thing the panel
	1548	+ * can count — and it was being handed the 4,000-character AI preview. Word
	1549	+ * Count, Keyword Density, Readability and Content Quality therefore
	1550	+ * described the first 4,000 characters and nothing after, freezing at
	1551	+ * roughly 600-700 words however long the page grew (#778).
	1552	+ *
	1553	+ * Still bounded, at a ceiling no real article reaches: this rides along on
	1554	+ * every editor page load.
	1555	+ *
	1556	+ * The body only, like the editor content it stands in for; the title and
	1557	+ * excerpt belong to the AI preview (see resolve_post_body()).
	1558	+ *
	1559	+ * @since 2.10.0
	1560	+ *
	1561	+ * @param \WP_Post $post Post object.
	1562	+ * @return string
	1563	+ */
	1564	+ public function get_analysis_content(\WP_Post $post): string {
	1565	+ return \ThinkRank\Core\Seo_Text::trim_to_length(
	1566	+ $this->resolve_post_body($post),
	1567	+ self::ANALYSIS_MAX_LENGTH
	1568	+ );
	1569	+ }
	1570	+
	1571	+ /**
	1572	+ * Get content preview for AI analysis
	1573	+ *
	1574	+ * @param \WP_Post $post Post object
	1575	+ * @return string Content preview
	1576	+ */
	1577	+ public function get_content_preview(\WP_Post $post): string {
	1578	+ // The title and excerpt lead the preview: they are context an AI
	1579	+ // caller wants first, and they are not part of the measured body.
	1580	+ $lead = $post->post_title . "\n\n";
	1581	+
	1582	+ if (!empty($post->post_excerpt)) {
	1583	+ $lead .= $post->post_excerpt . "\n\n";
	1584	+ }
	1585	+
	1586	+ $preview = trim(self::collapse_text($lead) . ' ' . $this->resolve_post_body($post));
	1587	+
	1588	+ // substr() counts BYTES: on Thai or CJK this handed the model a third
	1589	+ // of the intended content and cut the last character in half (#687).
	1590	+ return \ThinkRank\Core\Seo_Text::trim_to_length($preview, self::AI_PREVIEW_LENGTH);
	1591	+ }
	1592	+
	1593	+ /**
1247	1594	* AJAX handler for generating post metadata
1248	1595	*
1249	1596	* @return void
1250	1597	*/
		@@ -1302,8 +1649,22 @@
1302	1649	}
1303	1650
1304	1651	// phpcs:ignore WordPress.Security.NonceVerification.Missing, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized -- nonce verified above; each field sanitized inside persist_metadata()
1305	1652	$this->persist_metadata($post_id, wp_unslash($_POST));
	1653	+
	1654	+ // Everything else saved; only the redirect can have been refused. Report
	1655	+ // it in this response rather than as a notice on some later screen —
	1656	+ // this caller never reloads the page.
	1657	+ $redirect_error = $this->get_last_redirect_error();
	1658	+ if (null !== $redirect_error) {
	1659	+ wp_send_json_error([
	1660	+ 'message' => sprintf(
	1661	+ /* translators: %s: reason the redirect was not saved. */
	1662	+ __('Saved, except the redirect: %s', 'thinkrank'),
	1663	+ $redirect_error->get_error_message()
	1664	+ ),
	1665	+ ], 400);
	1666	+ }
1306	1667
1307	1668	wp_send_json_success([
1308	1669	'message' => __('SEO settings saved successfully!', 'thinkrank'),
1309	1670	]);