Skip to content

docs: add edge infrastructure concept page (Batch 3)#211

Open
marc0olo wants to merge 2 commits intoinfra/learn-hub-migration-prepfrom
docs/concepts-edge-infrastructure
Open

docs: add edge infrastructure concept page (Batch 3)#211
marc0olo wants to merge 2 commits intoinfra/learn-hub-migration-prepfrom
docs/concepts-edge-infrastructure

Conversation

@marc0olo
Copy link
Copy Markdown
Member

@marc0olo marc0olo commented May 6, 2026

Summary

  • Creates docs/concepts/edge-infrastructure.md covering API boundary nodes, HTTP gateways, the HTTP Gateway Protocol request lifecycle, and asset certification (certified variables for query responses)
  • Adds a "See also" link at the top of docs/references/http-gateway-spec.md
  • The HTTPS Outcalls staging file (skipped article) is left untouched

Staging files deleted

  • .migration/learn-hub/how-does-icp-work/icp-and-the-internet/icp-and-the-internet.mddocs/concepts/edge-infrastructure.md (intro section)
  • .migration/learn-hub/how-does-icp-work/icp-and-the-internet/icp-edge-infrastructure.mddocs/concepts/edge-infrastructure.md
  • .migration/learn-hub/how-does-icp-work/icp-and-the-internet/http-gateway-protocol.mddocs/concepts/edge-infrastructure.md#http-gateway-protocol
  • .migration/learn-hub/how-does-icp-work/icp-and-the-internet/asset-certification.mddocs/concepts/edge-infrastructure.md#asset-certification

Sync recommendation

hand-written

…gateway, asset certification)

Migrates 4 Learn Hub articles from the "ICP and the Internet" section into a
single docs/concepts/edge-infrastructure.md page covering API boundary nodes,
HTTP gateways, the HTTP Gateway Protocol request lifecycle, and asset
certification using chain-key cryptography and certified variables.

Adds a "See also" link at the top of docs/references/http-gateway-spec.md.
@marc0olo marc0olo requested a review from a team as a code owner May 6, 2026 14:59
@marc0olo
Copy link
Copy Markdown
Member Author

marc0olo commented May 6, 2026

Review notes

This PR is clean. All internal links resolve (http-gateway-spec.md, certified-variables.md, chain-key-cryptography.md), the bidirectional cross-reference with http-gateway-spec.md is correctly set, frontmatter is complete, and <!-- Upstream: --> comment is present. No banned patterns found.

Ready to merge.

@marc0olo
Copy link
Copy Markdown
Member Author

marc0olo commented May 6, 2026

Two issues to address before merging:

Em-dashes (banned project-wide)

docs/concepts/edge-infrastructure.md, lines 65–67 ("Further reading" bullets): three instances of need to be replaced with : or parentheses.

docs/references/http-gateway-spec.md, "See also" line added by this PR: one instance of needs the same treatment.

Per CLAUDE.md, em-dashes are banned in all prose, including bullet descriptions and "Further reading" / "See also" sections.

Missing section anchor

docs/concepts/edge-infrastructure.md, line 55: the link to chain-key-cryptography.md should be chain-key-cryptography.md#threshold-bls-signatures. The surrounding context is specifically about threshold BLS signatures, so a more precise destination is required per the project linking rules (always include a #anchor when the context refers to a specific subsection of the target page).

@marc0olo
Copy link
Copy Markdown
Member Author

marc0olo commented May 6, 2026

Feedback addressed:

  • Replaced 3 em-dashes with colons in edge-infrastructure.md Further reading bullets (lines 65–67)
  • Replaced em-dash with colon in http-gateway-spec.md See also line (line 8)
  • Added #threshold-bls-signatures anchor to the chain-key-cryptography.md link at line 55 of edge-infrastructure.md (context is specifically about threshold BLS signatures)

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