The latest updates on your projects. Learn more about Vercel for GitHub.

4 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
agui-chat-deepcitation	Ignored	Preview	Apr 2, 2026 0:26am
deepcitation-langchain-rag-chat	Ignored	Preview	Apr 2, 2026 0:26am
mastra-rag-deepcitation	Ignored	Preview	Apr 2, 2026 0:26am
nextjs-ai-sdk-deepcitation	Ignored	Preview	Apr 2, 2026 0:26am

✅ Playwright Test Report

Status: Tests passed

📊 Download Report & Snapshots (see Artifacts section)

Run ID: 23900267378

Code Review

Overall this is a solid PR — the proxy fix rationale is well-explained, the meta strip is clean, and the new tests are a meaningful addition. A few issues worth addressing before merge:

Bug / Correctness

http:// URLs silently rendered as links

buildMetaStrip uses /^https?:\/\//i to decide whether to render sourceUrl as a <a> tag, so plain http:// URLs are also linkified. The PR description and the JSDoc on sourceUrl both say "clickable link if https URL", and the test is named "falls back to sourceLabel when sourceUrl is not https" — but neither the test nor the description accounts for http:// being accepted.

Either the regex should be /^https:\/\//i (https-only as documented), or the docs/test names should be updated to say "http/https". The current state is a discrepancy between documentation and implementation.

Orphaned JSDoc comment

In markdownToHtml.ts, two consecutive JSDoc blocks appear before formatSourceUrl:

/**
 * Build the header meta strip: SOURCE · ANALYZED · AUDIENCE · CITATIONS · PAGES.
 * Only renders items that have data. Date always renders (defaults to today).
 */
/** Strip scheme from a URL for display: "https://example.com/doc" → "example.com/doc". */
function formatSourceUrl(url: string): string {

Only the second /** ... */ block attaches to formatSourceUrl — the first is an orphaned comment that never documents buildMetaStrip. Move the first comment directly above buildMetaStrip.

formatSourceUrl silently drops query strings and fragments

return u.hostname + (u.pathname !== "/" ? u.pathname : "");

This strips ?section=2#heading from URLs like https://example.com/doc?section=2#heading. For document URLs that use query params to identify pages/sections (e.g. Confluence, Google Docs), the rendered display text will be incomplete. If intentional for brevity, document it in a comment.

Cowork notice: inline styles vs. CSS class

plainShell renders the cowork notice with inline style="..." attributes; reportShell uses the .dc-cowork-notice CSS class. For reportShell this is fine, but plainShell will be harder to maintain if the notice styling ever needs to change. Consider adding the .dc-cowork-notice rule to BASE_CSS and using the class in both shells, or at minimum add a comment noting the intentional divergence.

sendViaUndiciProxy — silent fallback from ProxyAgent to EnvHttpProxyAgent

try {
  agent = new undici.ProxyAgent(proxyUrl);
} catch {
  console.error("Warning: ProxyAgent construction failed, falling back to EnvHttpProxyAgent.");
  agent = new undici.EnvHttpProxyAgent();
}

If ProxyAgent throws, it's most likely due to an invalid/malformed proxyUrl. Falling back to EnvHttpProxyAgent silently ignores the given proxyUrl and reads a different source (HTTP_PROXY/HTTPS_PROXY env vars). This could mask proxy misconfigurations. Consider re-throwing — or at minimum logging the actual error so users can see why the explicit URL was rejected.

Missing test coverage

The following new options have no tests:

• pageCount in the meta strip (new field, no test)
• reportDate option (accepted but untested)
• When both sourceUrl (https) and sourceLabel are provided — URL should take precedence over label, but this is unverified

Nits

• formatSourceUrl doesn't include u.port in the display string, so https://example.com:8443/doc renders as example.com/doc. Likely fine, but worth a comment.
• Test name "falls back to sourceLabel when sourceUrl is not https" is inaccurate given that http:// is also accepted — rename to "falls back to sourceLabel when sourceUrl has an unsupported scheme".