Skip to content

docs: document well-known discovery workflow and exclude internal assets - #24

Merged
anwar3606 merged 2 commits into
mainfrom
docs/well-known-discovery
Sep 8, 2026
Merged

anwar3606 merged 2 commits into
mainfrom
docs/well-known-discovery

Conversation

@anwar3606

Copy link
Copy Markdown
Contributor

Summary

  • Documentation: Added Section 3 to README.md explaining the GLEIF & KERI .well-known discovery architecture, source asset layout (well-known-assets/), and compilation procedure using scripts/build-wellknown.py.
  • Jekyll Exclusions: Added internal build scripts (scripts), source trees (well-known-assets), scratch research files (dns-research-file.txt), internal explainer (WELL-KNOWN.md), and legacy folders (css, images, js) to the exclude list in _config.yml to prevent unintended public deployment.
  • Script Reference: Updated script docstring and command examples in scripts/build-wellknown.py to target https://provenant.net.

Copilot AI lite review requested due to automatic review settings September 8, 2026 15:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

_config.yml excludes css/, images/, and js/ even though site pages reference those assets, which would break the deployed site.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This PR documents the repository’s .well-known discovery generation workflow (GLEIF/KERI) and updates the Jekyll build configuration to avoid publishing internal source/build artifacts.

Changes:

  • Added a README section describing the .well-known asset source layout, generation script, and update procedure.
  • Expanded Jekyll _config.yml exclude entries to prevent publishing internal directories/files.
  • Updated scripts/build-wellknown.py docstring/example command to target https://provenant.net.
File summaries
File Description
scripts/build-wellknown.py Updates generation-script documentation and example invocation host.
README.md Adds .well-known discovery workflow documentation and update instructions.
_config.yml Adjusts Jekyll exclude list to control what gets published to GitHub Pages.
Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread _config.yml Outdated
Comment thread README.md
Copilot AI review requested due to automatic review settings September 8, 2026 15:17
@anwar3606
anwar3606 marked this pull request as ready for review September 8, 2026 15:19
@anwar3606
anwar3606 merged commit 167a9c2 into main Sep 8, 2026
1 check passed
@anwar3606
anwar3606 deleted the docs/well-known-discovery branch September 8, 2026 15:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The well-known build script still defaults --host to https://weboftrust.github.io, which can generate incorrect host-meta.json output if the flag is omitted.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details
  • Files reviewed: 10/30 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread scripts/build-wellknown.py
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.

2 participants