@chenpengfei/daily-brief 0.1.1 → 0.1.3

package/CHANGELOG.md

@@ -2,6 +2,49 @@
 
 All notable changes for Formal Releases are recorded here. GitHub Release notes should be derived from the matching version entry.
 
+## 0.1.3 - 2026-06-06
+
+Patch release for runtime configuration hardening and richer local status output.
+
+### User-visible Changes
+
+- Expands `daily-brief status` into a local readiness report with setup checks, today's run state, active paths, system timezone, and the next suggested action.
+- Keeps model and Discord secrets in `auth.json` referenced by `config.yaml`, removing runtime reliance on `env:NAME`, `.env`, and `DISCORD_WEBHOOK_URL` credential paths for installed usage.
+- Reports Discord delivery as explicitly skipped when delivery is disabled or the stored webhook credential is missing.
+- Keeps `daily-brief run-once --date YYYY-MM-DD` available for manual backfills while making `daily-brief status` a no-flag inspection command.
+
+### Installation and Upgrade Notes
+
+- Upgrade with `npm install -g @chenpengfei/daily-brief@latest`.
+- Run `daily-brief setup` after upgrading if your previous model or Discord configuration used environment-backed credential references.
+- Scripted environments should write `config.yaml`, `sources.yaml`, and `auth.json` directly under `DAILY_BRIEF_HOME`; use `DAILY_BRIEF_DATA_HOME` when generated data should live elsewhere.
+
+### Known Limitations
+
+- `daily-brief status` is read-only and local: it does not collect Sources, call an LLM, refresh credentials, generate a brief, or send Discord notifications.
+- Environment variables remain limited to path overrides for installed usage: `DAILY_BRIEF_HOME` and `DAILY_BRIEF_DATA_HOME`.
+
+## 0.1.2 - 2026-06-04
+
+Patch release for the simplified public CLI setup workflow.
+
+### User-visible Changes
+
+- Simplifies the public CLI around `setup`, `run-once`, `status`, `sources`, `version`, and help.
+- Reworks `daily-brief setup` into an interactive wizard that preserves existing files by default and guides model credential and optional Discord delivery configuration.
+- Adds human-readable `run-once` progress output and clearer Source listing/edit guidance.
+- Makes `delivery.enabled: false` take precedence over `DISCORD_WEBHOOK_URL`, preventing accidental Discord sends when delivery is disabled in config.
+
+### Installation and Upgrade Notes
+
+- Upgrade with `npm install -g @chenpengfei/daily-brief@latest`.
+- Run `daily-brief setup` again after upgrading to review the simplified setup flow.
+- Non-interactive setup now fails before writing configuration files; scripted environments should write `config.yaml`, `sources.yaml`, and `auth.json` directly.
+
+### Known Limitations
+
+- The simplified public CLI removes direct public `collect`, `generate`, `deliver`, `model`, and `delivery` command entry points. Use `setup`, `run-once`, `status`, and `sources` for normal installed usage.
+
 ## 0.1.1 - 2026-06-01
 
 Patch release for first-install command discovery.

package/README.md

@@ -25,8 +25,7 @@ Daily Brief requires Node.js 22 or newer.
 daily-brief run-once
 daily-brief status
 daily-brief sources list
-daily-brief model status
-daily-brief delivery status
+daily-brief version
 ```
 
 For installation, setup, configuration, upgrade, and troubleshooting, see `docs/user-manual.md`.

package/dist/src/agent/daily-brief-agent.js

@@ -17,6 +17,7 @@ export async function runOnce(options = {}) {
     const dateKey = options.dateKey;
     let collection;
     try {
+        options.onProgress?.("1/5 Collecting Source Items");
         collection = await collectSources({
             date,
             ...(dateKey ? { dateKey } : {}),
@@ -24,6 +25,9 @@ export async function runOnce(options = {}) {
             ...(options.sourceRegistryPath ? { sourceRegistryPath: options.sourceRegistryPath } : {}),
             ...(options.sourceItemRoot ? { sourceItemRoot: options.sourceItemRoot } : {})
         });
+        for (const source of collection.sources) {
+            options.onProgress?.(`- ${source.sourceId}: ${source.status}, items ${source.itemCount}, written ${source.writtenCount}, duplicates ${source.skippedDuplicateCount}${source.reason ? `, ${source.reason}` : ""}`);
+        }
     }
     catch (error) {
         const coreFailure = {
@@ -32,6 +36,7 @@ export async function runOnce(options = {}) {
         };
         const delivery = await deliverCoreFailureNotification(coreFailure, {
             ...(options.discordWebhookUrl ? { webhookUrl: options.discordWebhookUrl } : {}),
+            ...(options.discordEnv ? { env: options.discordEnv } : {}),
             ...(options.discordFetchImpl ? { fetchImpl: options.discordFetchImpl } : {})
         });
         return {
@@ -50,6 +55,7 @@ export async function runOnce(options = {}) {
     if (collectionFailure) {
         const delivery = await deliverCoreFailureNotification(collectionFailure, {
             ...(options.discordWebhookUrl ? { webhookUrl: options.discordWebhookUrl } : {}),
+            ...(options.discordEnv ? { env: options.discordEnv } : {}),
             ...(options.discordFetchImpl ? { fetchImpl: options.discordFetchImpl } : {})
         });
         return {
@@ -64,6 +70,7 @@ export async function runOnce(options = {}) {
         };
     }
     const collectionFailures = collectionFailureRefs(collection);
+    options.onProgress?.("2/5 Generating Daily Brief");
     const generated = await generateOnce({
         ...options,
         ...(dateKey ? { dateKey } : {}),
@@ -75,6 +82,7 @@ export async function runOnce(options = {}) {
             }
             : {})
     });
+    options.onProgress?.("4/5 Delivering Notification");
     const delivery = await deliverOnce({
         ...options,
         date,
@@ -124,6 +132,7 @@ export async function generateOnce(options = {}) {
         const collectionInputRefs = options.collectionFailures
             ? { collectionFailures: options.collectionFailures }
             : undefined;
+        options.onProgress?.("- Understanding Source Items");
         const understanding = await runSourceItemUnderstandingStage({
             sourceItems,
             modelRuntimeConfig,
@@ -131,6 +140,7 @@ export async function generateOnce(options = {}) {
             ...(collectionInputRefs ? { inputRefs: collectionInputRefs } : {}),
             ...(options.modelRuntimeEnv ? { modelRuntimeEnv: options.modelRuntimeEnv } : {})
         });
+        options.onProgress?.("- Selecting and Ranking Signals");
         const selected = await runSignalSelectionAndRankingStages({
             sourceItems,
             annotations: understanding.annotations,
@@ -147,6 +157,7 @@ export async function generateOnce(options = {}) {
             signals: selected.signals
         };
         const narrativeEvents = [];
+        options.onProgress?.("- Writing Narrative");
         const narrative = await enrichDailyBriefNarrativeWithAgent({
             brief: selectedBrief,
             sourceItems,
@@ -161,6 +172,7 @@ export async function generateOnce(options = {}) {
             brief: narrative.brief,
             ...(options.collectionFailures ? { collectionFailures: options.collectionFailures } : {})
         });
+        options.onProgress?.("- Checking Source Grounding");
         const audited = await auditBriefWithSingleRepairAttempt({
             narrativeBrief: narrative.brief,
             sourceItems,
@@ -175,6 +187,7 @@ export async function generateOnce(options = {}) {
         const writtenArtifact = options.agentRunRoot ? await writeAgentRunArtifact(artifact, date, options.agentRunRoot, dateKey) : undefined;
         const markdown = renderDailyBriefMarkdown(audited.brief);
         const piResult = await renderBriefThroughPiRuntime(markdown);
+        options.onProgress?.("3/5 Archiving Daily Brief");
         const archived = await writeBriefArchive(piResult.markdown, date, options.archiveRoot, dateKey);
         return {
             archivePath: archived.path,
@@ -309,6 +322,7 @@ export async function deliverOnce(options = {}) {
         ...(options.discordTemplatePath ? { templatePath: options.discordTemplatePath } : {})
     }, {
         ...(options.discordWebhookUrl ? { webhookUrl: options.discordWebhookUrl } : {}),
+        ...(options.discordEnv ? { env: options.discordEnv } : {}),
         ...(options.discordFetchImpl ? { fetchImpl: options.discordFetchImpl } : {})
     });
 }

package/dist/src/agent/model-runtime-config.js

@@ -1,6 +1,6 @@
 import { existsSync } from "node:fs";
 import { getOAuthApiKey, loginOpenAICodex } from "@earendil-works/pi-ai/oauth";
-import { envNameFromCredentialRef, getCredential, isEnvCredentialRef, putCredential, readCredentialStore, readModelConfig, removeCredential, writeCredentialStore } from "../config/index.js";
+import { getCredential, putCredential, readCredentialStore, readModelConfig, removeCredential, writeCredentialStore } from "../config/index.js";
 import { resolveDailyBriefPaths } from "../config/paths.js";
 export const defaultOAuthHelpers = {
     getOAuthApiKey,
@@ -10,17 +10,17 @@ export function readModelRuntimeConfig(env = process.env, options = {}) {
     const paths = resolveDailyBriefPaths(env);
     const configPath = options.configPath ?? paths.configPath;
     const authPath = options.authPath ?? paths.authPath;
-    const config = readEnvModelConfig(env) ?? (existsSync(configPath) ? readModelConfig(configPath) : undefined);
+    const config = existsSync(configPath) ? readModelConfig(configPath) : undefined;
     if (!config) {
         return {
             provider: "openai-codex",
             model: "gpt-5.5",
             credentialRef: "openai-codex.default",
             ready: false,
-            issues: [`Model config not found: ${configPath}. Run daily-brief setup and daily-brief model configure/login first.`]
+            issues: [`Model config not found: ${configPath}. Run daily-brief setup or create config.yaml with model settings.`]
         };
     }
-    const issues = credentialIssues(config.provider, config.credentialRef, env, authPath);
+    const issues = credentialIssues(config.provider, config.credentialRef, authPath);
     return {
         provider: config.provider,
         model: config.model,
@@ -38,10 +38,6 @@ export async function resolveModelApiKey(config, env = process.env, options = {}
     if (!credentialRef) {
         throw new Error(`credentialRef is required for ${config.provider}`);
     }
-    if (isEnvCredentialRef(credentialRef)) {
-        const envName = envNameFromCredentialRef(credentialRef);
-        return env[envName]?.trim();
-    }
     const authPath = options.authPath ?? resolveDailyBriefPaths(env).authPath;
     const credential = getCredential(credentialRef, authPath);
     if (!credential) {
@@ -108,17 +104,13 @@ export function statusModelCredentials(env = process.env, options = {}) {
         secret: "<redacted>"
     }));
 }
-function credentialIssues(provider, credentialRef, env, authPath) {
+function credentialIssues(provider, credentialRef, authPath) {
     if (provider === "faux") {
         return [];
     }
     if (!credentialRef) {
         return [`credentialRef is required for ${provider}`];
     }
-    if (isEnvCredentialRef(credentialRef)) {
-        const envName = envNameFromCredentialRef(credentialRef);
-        return env[envName]?.trim() ? [] : [`${envName} is required for credentialRef ${credentialRef}`];
-    }
     const credential = getCredential(credentialRef, authPath);
     if (!credential) {
         return [`Credential not found: ${credentialRef}`];
@@ -128,65 +120,6 @@ function credentialIssues(provider, credentialRef, env, authPath) {
     }
     return [];
 }
-function readEnvModelConfig(env) {
-    const provider = env.DAILY_BRIEF_MODEL_PROVIDER?.trim();
-    if (!provider) {
-        return undefined;
-    }
-    const normalizedProvider = normalizeProvider(provider, env);
-    return {
-        provider: normalizedProvider,
-        model: env.DAILY_BRIEF_MODEL?.trim() || defaultModelForProvider(normalizedProvider),
-        ...(env.DAILY_BRIEF_MODEL_BASE_URL?.trim() ? { baseUrl: env.DAILY_BRIEF_MODEL_BASE_URL.trim() } : {}),
-        ...readEnvCredentialRef(env, normalizedProvider)
-    };
-}
-function normalizeProvider(value, env) {
-    const provider = value.trim().toLowerCase();
-    if (provider === "codex" || provider === "hermes") {
-        return "openai-codex";
-    }
-    if (provider === "faux") {
-        if (env.DAILY_BRIEF_ALLOW_FAUX_PROVIDER === "true") {
-            return provider;
-        }
-        throw new Error("DAILY_BRIEF_MODEL_PROVIDER=faux is only allowed when DAILY_BRIEF_ALLOW_FAUX_PROVIDER=true");
-    }
-    if (provider === "openai-codex" || provider === "openai" || provider === "deepseek" || provider === "openai-compatible") {
-        return provider;
-    }
-    throw new Error(`Unsupported DAILY_BRIEF_MODEL_PROVIDER: ${value}`);
-}
-function defaultModelForProvider(provider) {
-    if (provider === "openai-codex") {
-        return "gpt-5.5";
-    }
-    if (provider === "openai") {
-        return "gpt-4.1-mini";
-    }
-    if (provider === "deepseek") {
-        return "deepseek-chat";
-    }
-    if (provider === "openai-compatible") {
-        return "openai-compatible-model";
-    }
-    return "faux-daily-brief-renderer";
-}
-function defaultCredentialRefForProvider(provider) {
-    if (provider === "faux") {
-        return undefined;
-    }
-    if (provider === "openai") {
-        return "env:OPENAI_API_KEY";
-    }
-    if (provider === "deepseek") {
-        return "env:DEEPSEEK_API_KEY";
-    }
-    if (provider === "openai-compatible") {
-        return "env:OPENAI_API_KEY";
-    }
-    return "openai-codex.default";
-}
 function persistRefreshedOAuthCredential(ref, credential, credentials, authPath) {
     const store = readCredentialStore(authPath);
     const previous = store.credentials[ref];
@@ -198,11 +131,6 @@ function persistRefreshedOAuthCredential(ref, credential, credentials, authPath)
     };
     writeCredentialStore(store, authPath);
 }
-function readEnvCredentialRef(env, provider) {
-    const credentialRef = env.DAILY_BRIEF_MODEL_CREDENTIAL_REF?.trim() ||
-        defaultCredentialRefForProvider(provider);
-    return credentialRef ? { credentialRef } : {};
-}
 function promptForOAuth(io, message) {
     if (!io.prompt) {
         throw new Error(`OAuth login requires interactive input: ${message}`);