docs: add Oracle to README and docs - #434
Merged
Merged
Conversation
- ASCII architecture charts (README, docs/index) list Oracle after SQL Server; both stay 15 rows so the MCP client column is unchanged - Supported-database lists, mermaid overview, driver list (oracledb, Thin mode), quickstart card - Command-line: Oracle DSN tab (service name, ?sid=, TCPS), `sid` query parameter, SSL matrix row, verify-full wording, DB_TYPE and default port - TOML: individual-parameter example, connection/query timeout and sslmode support lists, `:1` placeholder row - Tools: explain_sql dialect note, health_check unsupported note, comments table row, custom-tools Oracle tab - package.json / server.json descriptions and the registry DB_TYPE enum include Oracle Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0189BHv585xi8iqEp9JvmgKY
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The documentation and metadata updates consistently match the implemented Oracle connector.
Review effort: Lite
Findings: None
What changed in this PR
Adds Oracle to user-facing documentation and package/MCP metadata, aligning docs with the implemented connector.
Changes:
- Updated architecture diagrams and supported-database lists.
- Documented Oracle installation, DSNs, SSL modes, TOML configuration, custom tools, and dialect behavior.
- Updated package and MCP Registry descriptions to include Oracle.
| File | Description |
|---|---|
server.json |
Adds Oracle to metadata descriptions. |
package.json |
Updates the package description. |
README.md |
Updates architecture and supported-database documentation. |
docs/index.mdx |
Updates overview diagrams and database lists. |
docs/installation.mdx |
Documents the Oracle driver. |
docs/quickstart.mdx |
Adds Oracle to DSN guidance. |
docs/config/command-line.mdx |
Documents Oracle DSNs, SSL, SID, and environment settings. |
docs/config/toml.mdx |
Adds Oracle TOML examples and supported options. |
docs/tools/search-objects.mdx |
Documents Oracle comments syntax. |
docs/tools/health-check.mdx |
Documents Oracle’s unsupported status. |
docs/tools/explain-sql.mdx |
Documents Oracle explain-plan behavior. |
docs/tools/custom-tools.mdx |
Adds an Oracle custom-tool example. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
tianzhou
changed the title
tianzhou
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #433, which added the Oracle connector but left the user-facing docs behind.
Changes
README.mdanddocs/index.mdxlist Oracle after SQL Server. Both charts keep their original height, so the MCP client column is untouched; MariaDB moves into the row that was blank.oracledbadded to the driver list, with a note that Thin mode needs no Oracle Instant Client.?sid=, TCPS), thesidquery parameter, an Oracle row in the SSL matrix,verify-fullwording (PostgreSQL and Oracle;sslrootcertstays PostgreSQL-only),DB_TYPEvalues, and the default port 1521.connection_timeout/query_timeout/sslmodesupport lists, and the:1placeholder row.EXPLAIN PLAN+DBMS_XPLAN), health_check listed as unsupported on Oracle,COMMENT ONrow in the comments table, and an Oracle tab in custom tools.package.jsonandserver.jsondescriptions, plus the MCP RegistryDB_TYPEenum. The manifest consistency test passes.Not changed
🤖 Generated with Claude Code
https://claude.ai/code/session_0189BHv585xi8iqEp9JvmgKY