Skip to content

fix(nx-dev): keep the kb index working for uncommitted articles - #36588

Merged
jaysoo merged 1 commit into
masterfrom
fix/kb-last-modified-uncommitted
Aug 6, 2026
Merged

fix(nx-dev): keep the kb index working for uncommitted articles#36588
jaysoo merged 1 commit into
masterfrom
fix/kb-last-modified-uncommitted

Conversation

@jaysoo

@jaysoo jaysoo commented Aug 6, 2026

Copy link
Copy Markdown
Member

Current Behavior

One uncommitted article takes down the whole Knowledge Base index. The batched git log has no entry for it, and the starlight fallback throws instead of returning a date, so /docs/kb 500s and astro-docs:validate-links fails.

Hits anyone drafting a new KB page.

Expected Behavior

Uncommitted articles date to now. Committed articles keep their commit date.

Related Issue(s)

N/A


Polygraph View session ↗

## Current Behavior

One uncommitted article takes down the whole Knowledge Base index. The batched
git log has no entry for it, and the starlight fallback throws instead of
returning a date, so `/docs/kb` 500s and `astro-docs:validate-links` fails.

Hits anyone drafting a new KB page.

## Expected Behavior

Uncommitted articles date to now. Committed articles keep their commit date.

## Related Issue(s)

N/A
@netlify

netlify Bot commented Aug 6, 2026

Copy link
Copy Markdown

Deploy Preview for nx-dev ready!

Name Link
🔨 Latest commit 95d876a
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/6a7481aed8f1ab000862d87a
😎 Deploy Preview https://deploy-preview-36588--nx-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Aug 6, 2026

Copy link
Copy Markdown

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 95d876a
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/6a7481ae1e56520008e9c3fd
😎 Deploy Preview https://deploy-preview-36588--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@polygraph-snapshot-app
polygraph-snapshot-app Bot marked this pull request as ready for review August 6, 2026 12:44
@polygraph-snapshot-app
polygraph-snapshot-app Bot requested a review from a team as a code owner August 6, 2026 12:44
@polygraph-snapshot-app
polygraph-snapshot-app Bot requested a review from lourw August 6, 2026 12:44
@nx-cloud

nx-cloud Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

View your CI Pipeline Execution ↗ for commit 95d876a

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 10m 42s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 4s View ↗
nx-cloud record -- pnpm nx-cloud conformance:check ✅ Succeeded 41s View ↗
nx build workspace-plugin ✅ Succeeded <1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 17s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 3s View ↗

☁️ Nx Cloud last updated this comment at 2026-08-06 12:59:45 UTC

@jaysoo
jaysoo merged commit e056ef6 into master Aug 6, 2026
26 checks passed
@jaysoo
jaysoo deleted the fix/kb-last-modified-uncommitted branch August 6, 2026 14:52
polygraph-snapshot-app Bot pushed a commit that referenced this pull request Aug 11, 2026
## Current Behavior

One uncommitted article takes down the whole Knowledge Base index. The
batched `git log` has no entry for it, and the starlight fallback throws
instead of returning a date, so `/docs/kb` 500s and
`astro-docs:validate-links` fails.

Hits anyone drafting a new KB page.

## Expected Behavior

Uncommitted articles date to now. Committed articles keep their commit
date.

## Related Issue(s)

N/A

<!-- polygraph-session-start -->
---
<p><picture><source media="(prefers-color-scheme: dark)"
srcset="https://static.ops.cloud.nx.app/polygraph/session-logo-v4-dark.svg"><img
src="https://static.ops.cloud.nx.app/polygraph/session-logo-v4-light.svg"
width="16" height="22" align="middle" alt="Polygraph"></picture> <a
href="https://snapshot.app.trypolygraph.com/orgs/69cdc268b6aa527e4129c2b4/sessions/vivid-moose-c9937bd5">View
session ↗</a></p>
<!-- polygraph-session-end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants