Skip to content

docs: correct architecture docs after the platform->infrastructure rename - #5327

Merged
YauhenBichel merged 1 commit into
mainfrom
docs/architecture-post-rename-cleanup
Aug 21, 2026
Merged

docs: correct architecture docs after the platform->infrastructure rename#5327
YauhenBichel merged 1 commit into
mainfrom
docs/architecture-post-rename-cleanup

Conversation

@YauhenBichel

Copy link
Copy Markdown
Collaborator

What

Fixes accuracy gaps in the architecture docs that remained after the
platform -> infrastructure rename (#5295) and the gateway turn-handler
unification. Documentation only — no code changes.

Why

An audit of the architecture docs against the current code found the rename was
almost fully propagated, but a few concrete defects and gaps survived: a wrong
tier count, a stale mermaid identifier, an undocumented core subpackage, and a
peer rule that no longer matches the enforced import graph.

@github-actions

Copy link
Copy Markdown
Contributor

Greptile code review

This repo uses Greptile for automated review. Before merge, aim for Confidence Score: 5/5 with zero unresolved review threads — see CONTRIBUTING.md.

Run a review — add a PR comment with:

@greptile review

Give it ~5-10 minutes (sometimes longer) for results, then fix feedback and re-trigger until you reach Confidence Score: 5/5.

Optional: automate with the greploop skill.

@YauhenBichel

Copy link
Copy Markdown
Collaborator Author

@greptile review

@greptile-apps

greptile-apps Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR updates architecture documentation after the platform to infrastructure rename and shared turn-handler unification.

  • Corrects the documented layer count from four to five.
  • Replaces the stale Mermaid PLATFORM identifier with INFRA.
  • Documents gateway transports, middleware, web responsibilities, and infrastructure/turn_host.
  • Records the three tracked tier-1 import exceptions.

Confidence Score: 5/5

The documentation-only PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
AGENTS.md Corrects the architecture-document description to identify a five-tier layer table.
docs/ARCHITECTURE.md Updates renamed infrastructure identifiers and documents current gateway, middleware, turn-host, and import-boundary architecture.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    HOSTS["Tier 1: surfaces + gateway"] --> BOOT["Tier 2: bootstrap"]
    BOOT --> CAP["Tier 3: tools + integrations"]
    CAP --> RUNTIME["Tier 4: core + infrastructure"]
    RUNTIME --> CONFIG["Tier 5: config"]
    CORE[core] <--> INFRA[infrastructure]
Loading

Reviews (2): Last reviewed commit: "docs: correct architecture docs after th..." | Re-trigger Greptile

@YauhenBichel
YauhenBichel marked this pull request as ready for review August 21, 2026 10:08
@YauhenBichel
YauhenBichel merged commit 9838965 into main Aug 21, 2026
14 checks passed
@YauhenBichel
YauhenBichel deleted the docs/architecture-post-rename-cleanup branch August 21, 2026 10:11
@github-actions

Copy link
Copy Markdown
Contributor

🛸 Aliens watching our repo just upgraded @YauhenBichel's threat level to: do not engage — too competent. 👽


👋 Join us on Discord - OpenSRE : hang out, contribute, or hunt for features and issues. Everyone's welcome.

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