scientify 1.13.6 → 2.0.0

package/dist/src/tools/scientify-cron.js

@@ -1,13 +1,11 @@
 import { Type } from "@sinclair/typebox";
-import { buildStateScopeKey, normalizeDeliveryChannelOverride, resolveDeliveryTarget, } from "../research-subscriptions/delivery.js";
-import { parseSubscribeOptions } from "../research-subscriptions/parse.js";
+import { normalizeDeliveryChannelOverride } from "../research-subscriptions/delivery.js";
 import { createResearchSubscribeHandler, createResearchSubscriptionsHandler, createResearchUnsubscribeHandler, } from "../research-subscriptions.js";
-import { getIncrementalStateStatus, recordIncrementalPush } from "../literature/subscription-state.js";
 import { Result } from "./result.js";
 export const ScientifyCronToolSchema = Type.Object({
-    action: Type.Optional(Type.String({
-        description: 'Action: "upsert" | "list" | "remove". When omitted, tool infers action from parameters.',
-    })),
+    action: Type.String({
+        description: 'Action: "upsert" | "list" | "remove".',
+    }),
     scope: Type.Optional(Type.String({
         description: "Scope key for grouping jobs (e.g. user ID or thread ID). Default: global.",
     })),
@@ -50,15 +48,6 @@ export const ScientifyCronToolSchema = Type.Object({
     no_deliver: Type.Optional(Type.Boolean({
         description: "If true, run in background without push delivery.",
     })),
-    metadata_only: Type.Optional(Type.Boolean({
-        description: "If true, allow metadata-only reading (skip full-text-first strict default). Use only when user explicitly requests it.",
-    })),
-    language: Type.Optional(Type.String({
-        description: 'Optional output language hint: "zh", "en", or "auto".',
-    })),
-    run_now: Type.Optional(Type.Boolean({
-        description: "If true (upsert only), trigger one immediate run after job creation/update; for research tasks, also return a status_json snapshot.",
-    })),
     job_id: Type.Optional(Type.String({
         description: "Specific job id to remove (only used when action=remove).",
     })),
@@ -72,78 +61,6 @@ function readStringParam(params, key) {
     const str = String(value).trim();
     return str.length > 0 ? str : undefined;
 }
-function sanitizeProjectId(raw) {
-    const trimmed = raw.trim();
-    if (/^[A-Za-z0-9_-]+$/.test(trimmed))
-        return trimmed;
-    const slug = trimmed
-        .toLowerCase()
-        .replace(/[^a-z0-9_-]+/g, "-")
-        .replace(/-+/g, "-")
-        .replace(/^-|-$/g, "");
-    return slug || "project";
-}
-function normalizeScheduleInput(raw, runNow) {
-    if (!raw)
-        return undefined;
-    const trimmed = raw.trim();
-    if (!trimmed)
-        return undefined;
-    const lower = trimmed.toLowerCase();
-    if (["now", "immediate", "immediately", "right now", "asap", "立即", "马上", "立刻"].includes(lower)) {
-        // run_now already triggers immediate execution; keep a valid persistent schedule.
-        return runNow ? "daily 09:00 Asia/Shanghai" : "at 2m";
-    }
-    if (/^\d+[smhdw]$/i.test(trimmed)) {
-        return `at ${trimmed}`;
-    }
-    if (/^every\s*hour$/i.test(trimmed) || /^每小时$/u.test(trimmed)) {
-        return "every 1h";
-    }
-    // Guard against `at <past-time>` generated by model/tool callers.
-    if (lower.startsWith("at ")) {
-        const when = trimmed.slice(3).trim();
-        if (when) {
-            const atMs = Date.parse(when);
-            if (!Number.isNaN(atMs) && atMs <= Date.now()) {
-                return runNow ? "daily 09:00 Asia/Shanghai" : "at 2m";
-            }
-        }
-    }
-    return trimmed;
-}
-function inferAction(params) {
-    const raw = readStringParam(params, "action")?.toLowerCase();
-    if (raw) {
-        if (["upsert", "create", "add", "set", "update", "start", "schedule", "new", "insert"].includes(raw)) {
-            return "upsert";
-        }
-        if (["list", "show", "ls", "status"].includes(raw)) {
-            return "list";
-        }
-        if (["remove", "delete", "cancel", "rm", "unsubscribe"].includes(raw)) {
-            return "remove";
-        }
-    }
-    const hasJobId = Boolean(readStringParam(params, "job_id"));
-    const hasUpsertSignals = Boolean(readStringParam(params, "schedule")) ||
-        Boolean(readStringParam(params, "topic")) ||
-        Boolean(readStringParam(params, "message")) ||
-        Boolean(readStringParam(params, "project")) ||
-        readBooleanParam(params, "run_now") ||
-        readBooleanParam(params, "no_deliver") ||
-        readBooleanParam(params, "metadata_only") ||
-        readNumberParam(params, "max_papers") !== undefined ||
-        readNumberParam(params, "recency_days") !== undefined ||
-        readNumberParam(params, "candidate_pool") !== undefined ||
-        Boolean(readStringParam(params, "channel")) ||
-        Boolean(readStringParam(params, "to"));
-    if (hasUpsertSignals)
-        return "upsert";
-    if (hasJobId)
-        return "remove";
-    return "list";
-}
 function readBooleanParam(params, key) {
     return params[key] === true;
 }
@@ -186,15 +103,6 @@ function normalizeScope(raw) {
         .replace(/^-|-$/g, "");
     return base || "global";
 }
-function parseJobIdFromResultText(text) {
-    const fenced = text.match(/Job ID:\s*`([^`]+)`/i);
-    if (fenced?.[1])
-        return fenced[1].trim();
-    const plain = text.match(/Job ID:\s*([a-z0-9-]{8,})/i);
-    if (plain?.[1])
-        return plain[1].trim();
-    return undefined;
-}
 function buildToolContext(scope, args, commandBody) {
     return {
         senderId: `tool_${scope}`,
@@ -203,13 +111,18 @@ function buildToolContext(scope, args, commandBody) {
         args,
         commandBody,
         config: {},
+        requestConversationBinding: async () => ({ status: "error", message: "not available in tool context" }),
+        detachConversationBinding: async () => ({ removed: false }),
+        getCurrentConversationBinding: async () => null,
     };
 }
 function getResultText(result) {
-    return result.text ?? result.error ?? "";
+    return result.text ?? "";
 }
 function getResultError(result) {
-    const maybe = result.error?.trim();
+    if (!result.isError)
+        return undefined;
+    const maybe = result.text?.trim();
     return maybe && maybe.length > 0 ? maybe : undefined;
 }
 function shouldPromoteMessageToTopic(message) {
@@ -235,272 +148,26 @@ function deriveTopicFromResearchMessage(message) {
     const normalized = text.trim();
     return normalized.length > 0 ? normalized : message.trim();
 }
-function resolveTopicAndMessage(params) {
+function buildSubscribeArgs(params) {
+    const parts = [];
+    const schedule = readStringParam(params, "schedule") ?? "daily 09:00 Asia/Shanghai";
+    parts.push(schedule);
     let topic = readStringParam(params, "topic");
     let message = readStringParam(params, "message");
     if (!topic && message && shouldPromoteMessageToTopic(message)) {
         topic = deriveTopicFromResearchMessage(message);
         message = undefined;
     }
-    return { topic, message };
-}
-function parseIncrementalScopeFromResultText(text) {
-    const fenced = text.match(/Incremental Scope:\s*`([^`]+)`/i);
-    if (fenced?.[1])
-        return fenced[1].trim();
-    const plain = text.match(/Incremental Scope:\s*([^\n]+)/i);
-    if (plain?.[1])
-        return plain[1].trim();
-    return undefined;
-}
-function latestRunId(status) {
-    return status?.recentChangeStats[0]?.runId;
-}
-function lastRunAtMs(status) {
-    return status?.knowledgeStateSummary?.lastRunAtMs ?? 0;
-}
-function lastPushedAtMs(status) {
-    return status?.lastPushedAtMs ?? 0;
-}
-function hasFreshRun(before, after, runStartedAtMs) {
-    const beforeRunId = latestRunId(before);
-    const afterRunId = latestRunId(after);
-    const hasFreshTimestamp = lastRunAtMs(after) >= runStartedAtMs || lastPushedAtMs(after) >= runStartedAtMs;
-    if (!before) {
-        return hasFreshTimestamp;
-    }
-    if (after.totalRuns > before.totalRuns)
-        return true;
-    if (afterRunId && beforeRunId && afterRunId !== beforeRunId)
-        return true;
-    if (!beforeRunId && afterRunId)
-        return true;
-    if (lastRunAtMs(after) > lastRunAtMs(before))
-        return true;
-    if (lastPushedAtMs(after) > lastPushedAtMs(before))
-        return true;
-    if (hasFreshTimestamp &&
-        lastRunAtMs(before) < runStartedAtMs &&
-        lastPushedAtMs(before) < runStartedAtMs) {
-        return true;
-    }
-    return false;
-}
-function uniqueScopeCandidates(candidates) {
-    const seen = new Set();
-    const result = [];
-    for (const candidate of candidates) {
-        if (!candidate)
-            continue;
-        const normalized = normalizeScope(candidate);
-        if (!normalized || seen.has(normalized))
-            continue;
-        seen.add(normalized);
-        result.push(normalized);
-    }
-    return result;
-}
-function buildFallbackRunId(jobId) {
-    const ts = new Date().toISOString().replace(/[-:.]/g, "").replace("T", "t").replace("Z", "z");
-    return `cron-${jobId}-${ts}-autofallback`;
-}
-function sleep(ms) {
-    return new Promise((resolve) => setTimeout(resolve, ms));
-}
-function parseCronRunMarker(raw) {
-    const text = (raw ?? "").trim();
-    if (!text)
-        return undefined;
-    try {
-        const parsed = JSON.parse(text);
-        return {
-            ok: typeof parsed.ok === "boolean" ? parsed.ok : undefined,
-            ran: typeof parsed.ran === "boolean" ? parsed.ran : undefined,
-            reason: typeof parsed.reason === "string" ? parsed.reason : undefined,
-        };
-    }
-    catch {
-        return undefined;
-    }
-}
-function serializeRunStatusSnapshot(status) {
-    const projectRecentPapers = status.knowledgeStateSummary?.recentPapers ?? [];
-    const globalById = new Map(status.recentPapers.map((paper) => [paper.id, paper]));
-    return {
-        scope: status.scope,
-        topic: status.topic,
-        topic_key: status.topicKey,
-        known_paper_count: status.knownPaperCount,
-        total_runs: status.totalRuns,
-        last_status: status.lastStatus ?? null,
-        last_pushed_at_ms: status.lastPushedAtMs ?? null,
-        latest_run_id: status.recentChangeStats[0]?.runId ?? null,
-        knowledge_state_summary: status.knowledgeStateSummary
-            ? {
-                project_id: status.knowledgeStateSummary.projectId,
-                stream_key: status.knowledgeStateSummary.streamKey,
-                run_profile: status.knowledgeStateSummary.runProfile,
-                total_runs: status.knowledgeStateSummary.totalRuns,
-                total_hypotheses: status.knowledgeStateSummary.totalHypotheses,
-                knowledge_topics_count: status.knowledgeStateSummary.knowledgeTopicsCount,
-                paper_notes_count: status.knowledgeStateSummary.paperNotesCount,
-                trigger_state: {
-                    consecutive_new_revise_days: status.knowledgeStateSummary.triggerState.consecutiveNewReviseDays,
-                    bridge_count_7d: status.knowledgeStateSummary.triggerState.bridgeCount7d,
-                    unread_core_backlog: status.knowledgeStateSummary.triggerState.unreadCoreBacklog,
-                    last_updated_at_ms: status.knowledgeStateSummary.triggerState.lastUpdatedAtMs,
-                },
-                quality_gate: {
-                    mode: status.knowledgeStateSummary.qualityGate.mode,
-                    severity: status.knowledgeStateSummary.qualityGate.severity,
-                    warnings: status.knowledgeStateSummary.qualityGate.warnings,
-                    fatal_reasons: status.knowledgeStateSummary.qualityGate.fatalReasons,
-                    blocking: status.knowledgeStateSummary.qualityGate.blocking,
-                    passed: status.knowledgeStateSummary.qualityGate.passed,
-                    full_text_coverage_pct: status.knowledgeStateSummary.qualityGate.fullTextCoveragePct,
-                    evidence_binding_rate_pct: status.knowledgeStateSummary.qualityGate.evidenceBindingRatePct,
-                    citation_error_rate_pct: status.knowledgeStateSummary.qualityGate.citationErrorRatePct,
-                    reasons: status.knowledgeStateSummary.qualityGate.reasons,
-                },
-                hypothesis_gate: {
-                    accepted: status.knowledgeStateSummary.hypothesisGate.accepted,
-                    rejected: status.knowledgeStateSummary.hypothesisGate.rejected,
-                    rejection_reasons: status.knowledgeStateSummary.hypothesisGate.rejectionReasons,
-                },
-                last_run_at_ms: status.knowledgeStateSummary.lastRunAtMs ?? null,
-                last_status: status.knowledgeStateSummary.lastStatus ?? null,
-                recent_hypotheses: status.knowledgeStateSummary.recentHypotheses.map((item) => ({
-                    id: item.id,
-                    statement: item.statement,
-                    trigger: item.trigger,
-                    created_at_ms: item.createdAtMs,
-                    file: item.file,
-                    strict_overall_score: typeof item.strictOverallScore === "number" ? item.strictOverallScore : null,
-                    strict_decision: item.strictDecision ?? null,
-                })),
-                last_reflection_tasks: status.knowledgeStateSummary.lastReflectionTasks,
-            }
-            : null,
-        recent_hypotheses: status.recentHypotheses.map((item) => ({
-            id: item.id,
-            statement: item.statement,
-            trigger: item.trigger,
-            created_at_ms: item.createdAtMs,
-            file: item.file,
-            strict_overall_score: typeof item.strictOverallScore === "number" ? item.strictOverallScore : null,
-            strict_decision: item.strictDecision ?? null,
-        })),
-        recent_change_stats: status.recentChangeStats.map((item) => ({
-            day: item.day,
-            run_id: item.runId,
-            new_count: item.newCount,
-            confirm_count: item.confirmCount,
-            revise_count: item.reviseCount,
-            bridge_count: item.bridgeCount,
-        })),
-        recent_papers: (projectRecentPapers.length > 0 ? projectRecentPapers : status.recentPapers).map((paper) => {
-            const paperId = typeof paper.id === "string" ? paper.id : "";
-            const fromGlobal = paperId ? globalById.get(paperId) : undefined;
-            return {
-                id: paperId || null,
-                title: paper.title ?? null,
-                url: paper.url ?? null,
-                last_score: "lastScore" in paper && typeof paper.lastScore === "number"
-                    ? paper.lastScore
-                    : "score" in paper && typeof paper.score === "number"
-                        ? paper.score
-                        : fromGlobal?.lastScore ?? null,
-                last_reason: "lastReason" in paper && typeof paper.lastReason === "string"
-                    ? paper.lastReason
-                    : "reason" in paper && typeof paper.reason === "string"
-                        ? paper.reason
-                        : fromGlobal?.lastReason ?? null,
-                first_pushed_at_ms: "firstPushedAtMs" in paper && typeof paper.firstPushedAtMs === "number"
-                    ? paper.firstPushedAtMs
-                    : fromGlobal?.firstPushedAtMs ?? null,
-                last_pushed_at_ms: "lastPushedAtMs" in paper && typeof paper.lastPushedAtMs === "number"
-                    ? paper.lastPushedAtMs
-                    : fromGlobal?.lastPushedAtMs ?? null,
-                push_count: "pushCount" in paper && typeof paper.pushCount === "number"
-                    ? paper.pushCount
-                    : fromGlobal?.pushCount ?? null,
-                full_text_read: "fullTextRead" in paper && typeof paper.fullTextRead === "boolean"
-                    ? paper.fullTextRead
-                    : null,
-                read_status: "readStatus" in paper && typeof paper.readStatus === "string"
-                    ? paper.readStatus
-                    : null,
-                unread_reason: "unreadReason" in paper && typeof paper.unreadReason === "string"
-                    ? paper.unreadReason
-                    : null,
-                evidence_anchors: "evidenceAnchors" in paper && Array.isArray(paper.evidenceAnchors)
-                    ? paper.evidenceAnchors.map((anchor) => ({
-                        section: anchor && typeof anchor === "object" && "section" in anchor && typeof anchor.section === "string"
-                            ? anchor.section
-                            : null,
-                        locator: anchor && typeof anchor === "object" && "locator" in anchor && typeof anchor.locator === "string"
-                            ? anchor.locator
-                            : null,
-                        claim: anchor && typeof anchor === "object" && "claim" in anchor && typeof anchor.claim === "string"
-                            ? anchor.claim
-                            : null,
-                        quote: anchor && typeof anchor === "object" && "quote" in anchor && typeof anchor.quote === "string"
-                            ? anchor.quote
-                            : null,
-                    }))
-                    : [],
-            };
-        }),
-        global_recent_papers: status.recentPapers.map((paper) => ({
-            id: paper.id,
-            title: paper.title ?? null,
-            url: paper.url ?? null,
-            last_score: paper.lastScore ?? null,
-            last_reason: paper.lastReason ?? null,
-            first_pushed_at_ms: paper.firstPushedAtMs,
-            last_pushed_at_ms: paper.lastPushedAtMs,
-            push_count: paper.pushCount,
-        })),
-        knowledge_state_missing_reason: status.knowledgeStateMissingReason ?? null,
-    };
-}
-function buildSubscribeArgs(params) {
-    const parts = [];
-    const schedule = normalizeScheduleInput(readStringParam(params, "schedule"), readBooleanParam(params, "run_now")) ??
-        "daily 09:00 Asia/Shanghai";
-    parts.push(schedule);
-    const resolved = resolveTopicAndMessage(params);
-    const topic = resolved.topic;
-    const message = resolved.message;
-    const scope = readStringParam(params, "scope");
     if (topic) {
         parts.push("--topic", quoteArg(topic));
     }
     const project = readStringParam(params, "project");
     if (project) {
-        parts.push("--project", quoteArg(sanitizeProjectId(project)));
+        parts.push("--project", quoteArg(project));
     }
     if (message) {
         parts.push("--message", quoteArg(message));
     }
-    const explicitLanguage = readStringParam(params, "language");
-    const languageCandidate = (() => {
-        if (explicitLanguage && ["zh", "en", "auto"].includes(explicitLanguage.toLowerCase())) {
-            return explicitLanguage.toLowerCase();
-        }
-        const channel = readStringParam(params, "channel");
-        if (channel && channel.toLowerCase() === "feishu")
-            return "zh";
-        if (scope && scope.toLowerCase().includes("feishu"))
-            return "zh";
-        const text = `${topic ?? ""} ${message ?? ""}`.trim();
-        if (/[\p{Script=Han}]/u.test(text))
-            return "zh";
-        return undefined;
-    })();
-    if (languageCandidate) {
-        parts.push("--language", quoteArg(languageCandidate));
-    }
     const maxPapers = readNumberParam(params, "max_papers");
     if (maxPapers !== undefined) {
         parts.push("--max-papers", String(Math.floor(maxPapers)));
@@ -538,13 +205,6 @@ function buildSubscribeArgs(params) {
     if (readBooleanParam(params, "no_deliver")) {
         parts.push("--no-deliver");
     }
-    if (readBooleanParam(params, "metadata_only")) {
-        parts.push("--metadata-only");
-    }
-    const language = readStringParam(params, "language");
-    if (language && ["zh", "en", "auto"].includes(language.toLowerCase())) {
-        parts.push("--language", quoteArg(language.toLowerCase()));
-    }
     return parts.join(" ");
 }
 export function createScientifyCronTool(deps) {
@@ -558,184 +218,19 @@ export function createScientifyCronTool(deps) {
         parameters: ScientifyCronToolSchema,
         execute: async (_toolCallId, rawArgs) => {
             const params = rawArgs;
-            const action = inferAction(params);
-            if (!action) {
-                return Result.err("invalid_params", 'Unable to infer action. Use one of: action="upsert" | "list" | "remove".');
-            }
+            const action = (readStringParam(params, "action") ?? "").toLowerCase();
             const scope = normalizeScope(readStringParam(params, "scope"));
             try {
                 if (action === "upsert") {
-                    // In tool context, delivery target may be unavailable unless caller explicitly sets channel/to.
-                    // Default to no-deliver when delivery is unspecified to avoid hard failure on creation.
-                    const hasDeliveryHints = Boolean(readStringParam(params, "channel")) || Boolean(readStringParam(params, "to"));
-                    const upsertParams = readBooleanParam(params, "no_deliver") || hasDeliveryHints ? params : { ...params, no_deliver: true };
-                    const args = buildSubscribeArgs(upsertParams);
+                    const args = buildSubscribeArgs(params);
                     const ctx = buildToolContext(scope, args, `/research-subscribe ${args}`);
-                    let expectedStateScopeKey;
-                    try {
-                        const parsed = parseSubscribeOptions(args);
-                        if (!("error" in parsed)) {
-                            const delivery = resolveDeliveryTarget(ctx, parsed);
-                            if (!("error" in delivery)) {
-                                expectedStateScopeKey = buildStateScopeKey(ctx, delivery);
-                            }
-                        }
-                    }
-                    catch {
-                        // keep best-effort behavior; fallback to parsed text scope or caller scope
-                    }
                     const res = await subscribe(ctx);
                     const err = getResultError(res);
                     if (err) {
                         return Result.err("operation_failed", err);
                     }
                     const text = getResultText(res);
-                    const jobId = parseJobIdFromResultText(text);
-                    const resolved = resolveTopicAndMessage(upsertParams);
-                    const incrementalScope = parseIncrementalScopeFromResultText(text);
-                    const project = readStringParam(upsertParams, "project");
-                    const runNow = readBooleanParam(upsertParams, "run_now");
-                    if (runNow && jobId) {
-                        const statusScopeCandidates = uniqueScopeCandidates([expectedStateScopeKey, incrementalScope, scope]);
-                        const beforeStatusByScope = new Map();
-                        if (resolved.topic) {
-                            for (const statusScope of statusScopeCandidates) {
-                                const before = await getIncrementalStateStatus({
-                                    scope: statusScope,
-                                    topic: resolved.topic,
-                                    ...(project ? { projectId: project } : {}),
-                                }).catch(() => undefined);
-                                beforeStatusByScope.set(statusScope, before);
-                            }
-                        }
-                        const runArgsPrimary = [
-                            "openclaw",
-                            "cron",
-                            "run",
-                            jobId,
-                            "--expect-final",
-                            "--timeout",
-                            "900000",
-                        ];
-                        const runStartedAtMs = Date.now();
-                        let runRes = await deps.runtime.system.runCommandWithTimeout(runArgsPrimary, {
-                            timeoutMs: 920_000,
-                        });
-                        if (runRes.code !== 0 &&
-                            /unknown option '--expect-final'|unknown option \"--expect-final\"|unknown option\s+--expect-final/i.test(runRes.stderr || "")) {
-                            // Backward compatibility for older OpenClaw versions.
-                            runRes = await deps.runtime.system.runCommandWithTimeout(["openclaw", "cron", "run", jobId], { timeoutMs: 600_000 });
-                        }
-                        let runAlreadyInProgress = false;
-                        if (runRes.code !== 0) {
-                            const marker = parseCronRunMarker(runRes.stdout) ?? parseCronRunMarker(runRes.stderr);
-                            if (marker?.ok === true && marker?.ran === false && marker?.reason === "already-running") {
-                                runAlreadyInProgress = true;
-                            }
-                            else {
-                                return Result.err("operation_failed", runRes.stderr || runRes.stdout || `cron run failed for job ${jobId}`);
-                            }
-                        }
-                        let statusSnapshot;
-                        if (resolved.topic) {
-                            try {
-                                let status;
-                                let statusScopeUsed;
-                                const deadline = Date.now() + (runAlreadyInProgress ? 300_000 : 120_000);
-                                while (Date.now() <= deadline) {
-                                    for (const statusScope of statusScopeCandidates) {
-                                        const fetched = await getIncrementalStateStatus({
-                                            scope: statusScope,
-                                            topic: resolved.topic,
-                                            ...(project ? { projectId: project } : {}),
-                                        }).catch(() => undefined);
-                                        const before = beforeStatusByScope.get(statusScope);
-                                        if (fetched && hasFreshRun(before, fetched, runStartedAtMs)) {
-                                            status = fetched;
-                                            statusScopeUsed = statusScope;
-                                            break;
-                                        }
-                                    }
-                                    if (status) {
-                                        break;
-                                    }
-                                    await sleep(1_000);
-                                }
-                                if (!status) {
-                                    const fallbackError = "run_now completed but no new persisted research run was detected. Auto-persisted fallback error run.";
-                                    try {
-                                        const fallbackPersistErrors = [];
-                                        for (const [idx, statusScope] of statusScopeCandidates.entries()) {
-                                            const persisted = await recordIncrementalPush({
-                                                scope: statusScope,
-                                                topic: resolved.topic,
-                                                ...(project ? { projectId: project } : {}),
-                                                status: "error",
-                                                runId: `${buildFallbackRunId(jobId)}-${idx + 1}`,
-                                                note: fallbackError,
-                                                papers: [],
-                                                knowledgeState: {
-                                                    corePapers: [],
-                                                    explorationPapers: [],
-                                                    explorationTrace: [],
-                                                    knowledgeChanges: [],
-                                                    knowledgeUpdates: [],
-                                                    hypotheses: [],
-                                                    runLog: {
-                                                        runProfile: readBooleanParam(upsertParams, "metadata_only") ? "fast" : "strict",
-                                                        error: "run_now completed but the agent turn did not persist via scientify_literature_state.record",
-                                                        notes: "Fallback persisted by scientify_cron_job guard to avoid stale status response.",
-                                                        tempCleanupStatus: "not_needed",
-                                                    },
-                                                },
-                                            }).catch((persistError) => {
-                                                fallbackPersistErrors.push(`${statusScope}:${persistError instanceof Error ? persistError.message : String(persistError)}`);
-                                                return undefined;
-                                            });
-                                            if (!persisted)
-                                                continue;
-                                            const fetched = await getIncrementalStateStatus({
-                                                scope: statusScope,
-                                                topic: resolved.topic,
-                                                ...(project ? { projectId: project } : {}),
-                                            }).catch(() => undefined);
-                                            const before = beforeStatusByScope.get(statusScope);
-                                            if (fetched && hasFreshRun(before, fetched, runStartedAtMs)) {
-                                                status = fetched;
-                                                statusScopeUsed = statusScope;
-                                                break;
-                                            }
-                                        }
-                                        if (!status) {
-                                            return Result.err("operation_failed", `${fallbackError} fresh_status_unavailable_after_fallback_persist. scopes=${statusScopeCandidates.join(",")} errors=${fallbackPersistErrors.join(" | ") || "none"}`);
-                                        }
-                                    }
-                                    catch (persistError) {
-                                        return Result.err("operation_failed", `run_now completed but no new persisted research run was detected. Refusing stale status response. fallback_persist_error=${persistError instanceof Error ? persistError.message : String(persistError)}`);
-                                    }
-                                }
-                                statusSnapshot = {
-                                    ...serializeRunStatusSnapshot(status),
-                                    status_scope_used: statusScopeUsed ?? null,
-                                };
-                            }
-                            catch (statusError) {
-                                statusSnapshot = {
-                                    error: statusError instanceof Error ? statusError.message : String(statusError),
-                                };
-                            }
-                        }
-                        return Result.ok({
-                            action,
-                            scope,
-                            job_id: jobId,
-                            run_now: true,
-                            run_status: runRes.code === 0 ? "ok" : "already-running",
-                            ...(statusSnapshot ? { status_json: statusSnapshot } : {}),
-                            result: text,
-                        });
-                    }
-                    return Result.ok({ action, scope, ...(jobId ? { job_id: jobId } : {}), result: text });
+                    return Result.ok({ action, scope, result: text });
                 }
                 if (action === "list") {
                     const ctx = buildToolContext(scope, "", "/research-subscriptions");