Conversation
## Current Behavior Pylon crawls nx.dev, but crawled training data does not feed the support widget's suggested-answer feature. Only real KB articles are eligible. The 173 articles left in Pylon from the closed #36277 migration are stale copies of pre-DOC-552 doc paths, with no way to refresh them. ## Expected Behavior Nightly job mirrors `astro-docs/src/content/docs/kb` into Pylon. nx.dev stays canonical - copy only, no source deletion, no redirects. - `markdoc-to-html.ts` converts the Markdoc AST to Pylon-safe HTML. All 184 articles convert with no warnings. - `pylon-client.ts` wraps the REST API with retry, throttle and pagination. - `sync-kb.ts` diffs source against live and creates/updates. Pylon stores `body_html` verbatim, so change detection is a byte compare - no state file. - Images carry a `data-nx-src` content-hash marker so re-runs reuse the uploaded CDN URL instead of re-uploading. - Writes are confined to one collection, so the hand-written enterprise articles beside them cannot be touched. - `--prune` is opt-in and refuses to delete more than a quarter of the collection. Articles stay unlisted to keep nx.dev SEO-canonical. Needs `PYLON_API_TOKEN` in CI. ## Related Issue(s) Fixes DOC-542
✅ Deploy Preview for nx-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for nx-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
## Current Behavior Workflow only runs nightly or via workflow_dispatch, which GitHub exposes only once the file is on the default branch. No way to exercise it from a PR. ## Expected Behavior Adds a pull_request trigger scoped to the pylon tooling and kb content, always forced to --dry-run so a review never writes to Pylon. - New preview job converts all articles with no credentials, so fork PRs get signal too. - Sync job skips on fork PRs, where secrets are unavailable. - workflow_dispatch dry-run now defaults to true. - Slack notification narrows to scheduled runs. ## Related Issue(s) DOC-542
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Current Behavior
Pylon crawls nx.dev, but crawled training data does not feed the support widget's suggested-answer feature - only real KB articles are eligible. The 173 articles left in Pylon from the closed #36277 migration are stale copies of pre-DOC-552 doc paths with no way to refresh them.
Expected Behavior
A nightly job mirrors
astro-docs/src/content/docs/kbinto Pylon. nx.dev stays canonical - copy only, no source deletion and no redirects.Pylon stores
body_htmlverbatim, so change detection is a byte compare with no state file. Images carry adata-nx-srccontent-hash marker so re-runs reuse the uploaded CDN URL. Writes are confined to one collection, keeping the hand-written enterprise articles out of reach, and--pruneis opt-in and refuses to delete more than a quarter of the collection.All 184 articles convert with no warnings. Articles stay unlisted so nx.dev remains SEO-canonical.
Still needed before the nightly can run:
PYLON_API_TOKENin repo secrets.Related Issue(s)
DOC-542