Skip to content

[Docs|Models] Mark what the model catalog replaces - #1900

Merged
CREDO23 merged 2 commits into
MODSetter:devfrom
CREDO23:docs/model-catalog
Sep 23, 2026
Merged

CREDO23 merged 2 commits into
MODSetter:devfrom
CREDO23:docs/model-catalog

Conversation

@CREDO23

@CREDO23 CREDO23 commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • The catalog, connections and selection pages now say at the top that the model catalog proposal replaces them, and describe today's code until it ships.
  • The proposal gains "What this replaces": each affected architecture page and its fate, and the four catalog Known gaps it closes.
  • A TODO in write_presets names why the startup warm-up is lost: every rewrite restarts the router mid-load.

Follows #1899.

🤖 Generated with Claude Code

High-level PR Summary

This PR adds documentation clarifications marking which architecture pages are being redesigned by the model catalog proposal. It updates four architecture documentation files (connections.md, catalog.md, and selection.md) to note they describe current code until the redesign ships, adds a comprehensive "What this replaces" section to the proposal explaining the migration path and known gaps being fixed, and documents a TODO about startup warm-up performance loss caused by configuration file rewrites.

⏱️ Estimated Review Time: 5-15 minutes

💡 Review Order Suggestion
Order File Path
1 docs/proposals/model-catalog.md
2 docs/architecture/connections.md
3 docs/architecture/local-models/catalog.md
4 docs/architecture/local-models/selection.md
5 surfsense_local/backend/modules/llm/providers/llamacpp/preset.py

Need help? Join our Discord

CREDO23 and others added 2 commits September 23, 2026 14:43
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 23, 2026

Copy link
Copy Markdown

@CREDO23 is attempting to deploy a commit to the Rohan Verma's projects Team on Vercel.

A member of the Team first needs to authorize it.

@CREDO23
CREDO23 merged commit 052744e into MODSetter:dev Sep 23, 2026
3 of 7 checks passed
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