Skip to content

docs: architecture doc accuracy pass#23

Merged
devonartis merged 3 commits into
developfrom
fix/architecture-doc-cleanup
Apr 13, 2026
Merged

docs: architecture doc accuracy pass#23
devonartis merged 3 commits into
developfrom
fix/architecture-doc-cleanup

Conversation

@devonartis

Copy link
Copy Markdown
Owner

Summary

  • Removed Prometheus branding from obs package descriptions (implementation detail, not a feature)
  • Fixed SQLite version in deps table: v1.35.0 → v1.46.1
  • Added missing cobra dependency to table
  • Fixed "Open Source" → "Free for internal use" in token-lifecycle SVG
  • No enterprise, resource server, monitoring, or HITL references in any diagram or doc section

Finishes the WIP from fix/architecture-doc-cleanup (commit 74ba4ff).

Test plan

  • go build ./... passes
  • go test ./... -short all packages pass
  • SVGs verified clean of enterprise refs
  • Mermaid diagrams match actual codebase packages
  • CI gates-passed green

- Remove Prometheus branding from obs package descriptions (impl detail)
- Fix SQLite version v1.35.0 → v1.46.1 (matches go.mod)
- Add cobra to External Dependencies table (was missing)
- Fix "Open Source" → "Free for internal use" in token-lifecycle SVG
- No enterprise, resource server, or monitoring refs in diagrams
@github-actions

github-actions Bot commented Apr 13, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 2ce23f5.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@devonartis devonartis merged commit 9f11c53 into develop Apr 13, 2026
20 checks passed
@devonartis devonartis deleted the fix/architecture-doc-cleanup branch April 13, 2026 18:45
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