Skip to content

chore: export only docs/ to the docs site - #5

Merged
hamidfzm merged 4 commits into
mainfrom
chore/docs-only-site
Aug 16, 2026
Merged

chore: export only docs/ to the docs site#5
hamidfzm merged 4 commits into
mainfrom
chore/docs-only-site

Conversation

@hamidfzm

Copy link
Copy Markdown
Member

The docs site at https://glyph-md.github.io/plugins/ currently publishes the whole repository as the workspace, so the nav tree shows registration folders, the repo README, and CONTRIBUTING next to the actual documentation.

  • docs-site.yml now exports docs/ only; its README becomes the site index
  • docs/.glyph/site.json (new) sets the site title, description, and base URL
  • scripts/build-index.mjs links each catalog entry to the plugin's folder on GitHub (relative registration pages no longer exist on the site); docs/plugin-catalog.md regenerated
  • plugins/** stays in the workflow path filter since registration changes regenerate the catalog

Note: doc page URLs move from /plugins/docs/<page> to /plugins/<page> when this merges; the plugin-template PR's README links are being updated to match.

hamidfzm added 4 commits July 20, 2026 17:03
The site published the whole repo (registration folders, repo README,
CONTRIBUTING), which cluttered the nav with non-doc pages. docs/ is now
the exported workspace: its README becomes the index, docs/.glyph/site.json
carries the site title, and the generated catalog links each plugin's
README on GitHub instead of a relative page that no longer exists.
The guard diffed index.json, index/, and docs/plugin-catalog.md against the
base branch, so any PR that changes scripts/build-index.mjs failed: its output
legitimately moves with it, which is exactly what this PR does to the catalog
links.

Run the generator and diff against the commit instead. Hand edits still fail,
because a hand-edited file does not reproduce; a generator change passes as
long as its committed output was regenerated.
@hamidfzm
hamidfzm merged commit 20e0f9b into main Aug 16, 2026
1 check passed
@hamidfzm hamidfzm self-assigned this Aug 16, 2026
@hamidfzm
hamidfzm deleted the chore/docs-only-site branch August 16, 2026 12:02
@hamidfzm

Copy link
Copy Markdown
Member Author

Pushed 4acb085 to unblock CI here, plus a merge of main.

The validate job was failing on this PR's own catalog change:

index.json, index/, and docs/plugin-catalog.md are generated from plugins/*/plugin.json.
docs/plugin-catalog.md | 6 +++---

The guard diffed the generated files against the base branch, which cannot distinguish a hand edit from a generator change. This PR changes scripts/build-index.mjs, so its output legitimately moves with it and the guard rejected it, which is why the job has been red since July.

It now runs node scripts/build-index.mjs and diffs against the commit instead. A hand-edited file still fails, because it does not reproduce; a generator change passes as long as the committed output was regenerated. validate is green.

Two follow-ups, both filed:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant