@scalemule/news-publication 0.0.94 → 0.0.95

package/bin/scalemule-article.mjs

@@ -26,7 +26,8 @@ Unspecified fields and media are retained. Default: save for review.
 APPROVE or changes a policy. Legacy onboarding requires an explicit
 --enable-review-policy value. Scheduled legacy articles stop before onboarding.
 Use --expected-generation when coordinating an edit with another editor.
-Default edit deadline: 10000 ms. Reuse the same patch/key/checkpoint on retry.
+Default edit deadline: 10000 ms (--timeout-ms, API work). The public page check after --release
+has its own --page-timeout-ms (default 60000). Reuse the same patch/key/checkpoint on retry.
 RELEASE_HELD means the revision is saved and existing review gates remain.
 
 replace-photo accepts a local file and uses normal member/API-key credentials.
@@ -58,7 +59,7 @@ that every website/CDN cache already displays the new image.
 try {
   const { values, positionals } = parseArgs({
     allowPositionals: true, strict: true,
-    options: Object.fromEntries(['publication', 'article', 'file-id', 'expected-version', 'metadata', 'application-id', 'api-url', 'url', 'file', 'operation-key', 'state', 'timeout-ms','patch','expected-generation','enable-review-policy','manifest','submission'].map(k => [k, { type: 'string' }]).concat([['asset',{type:'string',multiple:true}],['help', { type: 'boolean' }],['release',{type:'boolean'}]])),
+    options: Object.fromEntries(['publication', 'article', 'file-id', 'expected-version', 'metadata', 'application-id', 'api-url', 'url', 'file', 'operation-key', 'state', 'timeout-ms','page-timeout-ms','patch','expected-generation','enable-review-policy','manifest','submission'].map(k => [k, { type: 'string' }]).concat([['asset',{type:'string',multiple:true}],['help', { type: 'boolean' }],['release',{type:'boolean'}]])),
   });
   if (values.help || !positionals.length) {
     console.log(help);
@@ -68,6 +69,10 @@ try {
     const budget = Number(values['timeout-ms'] ?? (command==='edit'?10000:30000));
     if (!Number.isFinite(budget) || budget < 100 || budget > 120000) throw Error('--timeout-ms must be between 100 and 120000');
     const remaining = () => { const ms = Math.floor(budget - (performance.now()-commandStarted)); if (ms < 1) throw Error('Operation deadline reached; resume the same operation'); return ms; };
+    // The public page check has its own deadline: caches and rolling pods make it slower than the API, and
+    // a slow page must never consume the API operation's budget or vice versa.
+    const pageBudget = Number(values['page-timeout-ms'] ?? 60000);
+    if (!Number.isFinite(pageBudget) || pageBudget < 1000 || pageBudget > 300000) throw Error('--page-timeout-ms must be between 1000 and 300000');
     if (extra.length || !['submit','status','inspect', 'replace-cover', 'replace-photo','edit'].includes(command)) throw Error('Use submit, status, inspect, edit, replace-cover or replace-photo; see --help');
     if (values.url) {
       if (!process.env.SCALEMULE_API_KEY || !process.env.SCALEMULE_ACCESS_TOKEN || values['application-id']) throw Error('URL resolution requires normal publication member/API-key credentials');
@@ -152,15 +157,17 @@ try {
         });
         if(values.url&&values.release&&['published','already_published'].includes(result.state)){
           let verified=false,reason;
+          const pageStarted=performance.now();
+          const pageRemaining=()=>{const ms=Math.floor(pageBudget-(performance.now()-pageStarted));if(ms<1)throw Error('Public page deadline reached; the page may still be refreshing. Resume the same operation or raise --page-timeout-ms');return ms;};
           try{
-            const snapshot=await client.publicArticle(checkpoint.sourceIdentity.slug,AbortSignal.timeout(remaining()));
+            const snapshot=await client.publicArticle(checkpoint.sourceIdentity.slug,AbortSignal.timeout(pageRemaining()));
             const article=snapshot.article;
             if(!article||!article.content_html||article.id!==values.article)throw Error('Published reader snapshot is missing');
             while(!verified){
-              const response=await fetch(values.url,{redirect:'error',cache:'no-store',signal:AbortSignal.timeout(Math.min(5000,remaining()))});
+              const response=await fetch(values.url,{redirect:'error',cache:'no-store',signal:AbortSignal.timeout(Math.min(5000,pageRemaining()))});
               const html=await response.text();
-              verified=response.ok&&articleEditHtmlMatches(html,{title:article.title,headline:article.headline,content_html:article.content_html,card_titles:(article.metadata?.cards??[]).map(c=>c.title),tags:checkpoint.document.tag_ids?article.tags:undefined});
-              if(!verified)await new Promise(r=>setTimeout(r,Math.min(300,remaining())));
+              verified=response.ok&&articleEditHtmlMatches(html,{title:article.title,headline:article.headline,seo_title:'seo_title' in patch?article.seo_title:undefined,content_html:article.content_html,card_titles:(article.metadata?.cards??[]).map(c=>c.title),tags:checkpoint.document.tag_ids?article.tags:undefined});
+              if(!verified)await new Promise(r=>setTimeout(r,Math.min(1000,pageRemaining())));
             }
           }catch(error){reason=error.message;}
           if(!verified){console.error(JSON.stringify({...result,state:'published_public_verification_pending',stage:'public_page',message:'The exact revision is published; rendered prose/cards/tags remain unverified. Resume the same edit.',reason}));process.exitCode=2;}

package/dist/article-workflow-client.cjs

@@ -36,6 +36,11 @@ function articleEditHtmlMatches(html, expected) {
     (m) => text(m[1]) === shown
   );
   if (!headline) return false;
+  const seo = expected.seo_title?.trim();
+  if (seo) {
+    const pageTitle = text(visible.match(/<title\b[^>]*>([\s\S]*?)<\/title>/iu)?.[1] ?? "");
+    if (pageTitle !== plain(seo) && !pageTitle.startsWith(plain(seo) + " ")) return false;
+  }
   const paragraphs = [
     ...expected.content_html.matchAll(
       /<(p|li|h[2-6]|blockquote)\b[^>]*>([\s\S]*?)<\/\1>/giu
@@ -55,13 +60,10 @@ function articleEditHtmlMatches(html, expected) {
       }
     }))) return false;
   }
-  const titles = [
-    ...visible.matchAll(
-      /<aside\b[^>]*class=["'][^"']*\barticle-verified-card\b[^"']*["'][^>]*>([\s\S]*?)<\/aside>/giu
-    )
-  ].flatMap(
-    (m) => [...m[1].matchAll(/<h2\b[^>]*>([\s\S]*?)<\/h2>/giu)].map((m2) => text(m2[1]))
-  );
+  const titles = [...visible.matchAll(/<aside\b([^>]*)>([\s\S]*?)<\/aside>/giu)].filter((m) => /\bdata-entity-card=["'][^"']+["']/iu.test(m[1]) || /class=["'][^"']*\barticle-verified-card\b/iu.test(m[1])).flatMap((m) => {
+    const first = m[2].match(/<h2\b[^>]*>([\s\S]*?)<\/h2>/iu);
+    return first ? [text(first[1])] : [];
+  });
   return JSON.stringify(titles.sort()) === JSON.stringify(expected.card_titles.map(plain).sort());
 }
 
@@ -680,7 +682,7 @@ async function runArticleEdit(t, input) {
         409,
         state.revisionId
       );
-    for (const key of ["title", "subtitle", "content_markdown", "excerpt"])
+    for (const key of ["title", "headline", "fly_title", "seo_title", "subtitle", "content_markdown", "excerpt"])
       if (state.document?.[key] !== void 0 && !equal(after[key] ?? "", state.document[key]))
         throw new ArticleEditOperationError(
           "VERIFICATION_CONFLICT",

package/dist/article-workflow-client.d.cts

@@ -3,10 +3,17 @@ import { A as ArticleFormat, b as ArticleLayout } from './article-presentation-0
 /** Check rendered image tags, not a URL mentioned only in serialized page data. */
 declare function articlePhotoHtmlMatches(html: string, lead: string | string[], social: string): boolean;
 
-/** Requires visible body and actual card asides; serialized page data is not delivery. */
+/**
+ * Requires visible body and actual card asides; serialized page data is not delivery.
+ * Cards are recognised by either rendering the papers use: the package's `article-verified-card`
+ * aside, or the papers' entity cards (`<aside data-entity-card="venue|event|…">`), each titled by its h2.
+ * When the edit carried a search title, the document's `<title>` must show it (a site may append its
+ * own name after it).
+ */
 declare function articleEditHtmlMatches(html: string, expected: {
     title: string;
     headline?: string | null;
+    seo_title?: string | null;
     content_html: string;
     card_titles: string[];
     tags?: {

package/dist/article-workflow-client.d.ts

@@ -3,10 +3,17 @@ import { A as ArticleFormat, b as ArticleLayout } from './article-presentation-0
 /** Check rendered image tags, not a URL mentioned only in serialized page data. */
 declare function articlePhotoHtmlMatches(html: string, lead: string | string[], social: string): boolean;
 
-/** Requires visible body and actual card asides; serialized page data is not delivery. */
+/**
+ * Requires visible body and actual card asides; serialized page data is not delivery.
+ * Cards are recognised by either rendering the papers use: the package's `article-verified-card`
+ * aside, or the papers' entity cards (`<aside data-entity-card="venue|event|…">`), each titled by its h2.
+ * When the edit carried a search title, the document's `<title>` must show it (a site may append its
+ * own name after it).
+ */
 declare function articleEditHtmlMatches(html: string, expected: {
     title: string;
     headline?: string | null;
+    seo_title?: string | null;
     content_html: string;
     card_titles: string[];
     tags?: {

package/dist/article-workflow-client.js

@@ -36,6 +36,11 @@ function articleEditHtmlMatches(html, expected) {
     (m) => text(m[1]) === shown
   );
   if (!headline) return false;
+  const seo = expected.seo_title?.trim();
+  if (seo) {
+    const pageTitle = text(visible.match(/<title\b[^>]*>([\s\S]*?)<\/title>/iu)?.[1] ?? "");
+    if (pageTitle !== plain(seo) && !pageTitle.startsWith(plain(seo) + " ")) return false;
+  }
   const paragraphs = [
     ...expected.content_html.matchAll(
       /<(p|li|h[2-6]|blockquote)\b[^>]*>([\s\S]*?)<\/\1>/giu
@@ -55,13 +60,10 @@ function articleEditHtmlMatches(html, expected) {
       }
     }))) return false;
   }
-  const titles = [
-    ...visible.matchAll(
-      /<aside\b[^>]*class=["'][^"']*\barticle-verified-card\b[^"']*["'][^>]*>([\s\S]*?)<\/aside>/giu
-    )
-  ].flatMap(
-    (m) => [...m[1].matchAll(/<h2\b[^>]*>([\s\S]*?)<\/h2>/giu)].map((m2) => text(m2[1]))
-  );
+  const titles = [...visible.matchAll(/<aside\b([^>]*)>([\s\S]*?)<\/aside>/giu)].filter((m) => /\bdata-entity-card=["'][^"']+["']/iu.test(m[1]) || /class=["'][^"']*\barticle-verified-card\b/iu.test(m[1])).flatMap((m) => {
+    const first = m[2].match(/<h2\b[^>]*>([\s\S]*?)<\/h2>/iu);
+    return first ? [text(first[1])] : [];
+  });
   return JSON.stringify(titles.sort()) === JSON.stringify(expected.card_titles.map(plain).sort());
 }
 
@@ -652,7 +654,7 @@ async function runArticleEdit(t, input) {
         409,
         state.revisionId
       );
-    for (const key of ["title", "subtitle", "content_markdown", "excerpt"])
+    for (const key of ["title", "headline", "fly_title", "seo_title", "subtitle", "content_markdown", "excerpt"])
       if (state.document?.[key] !== void 0 && !equal(after[key] ?? "", state.document[key]))
         throw new ArticleEditOperationError(
           "VERIFICATION_CONFLICT",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scalemule/news-publication",
-  "version": "0.0.94",
+  "version": "0.0.95",
   "description": "Shared server framework for ScaleMule local news publications. Walnut Creek Times is the broadsheet default; each paper supplies its own identity and feature flags.",
   "type": "module",
   "bin": {