feat(sources/oracle): add Oracle Source and Tool

[vijaybalebail]

Description

---

Should include a concise description of the changes (bug or feature), it's
impact, along with a summary of the solution

PR Checklist

---

Thank you for opening a Pull Request! Before submitting your PR, there are a
few things you can do to make sure it goes smoothly:

• Make sure you reviewed
  CONTRIBUTING.md
• Make sure to open an issue as a
  bug/issue
  before writing your code! That way we can discuss the change, evaluate
  designs, and agree on the general idea
• Ensure the tests and linter pass
• Code coverage does not decrease (if any source code was changed)
• Appropriate docs were updated (if necessary)
• Make sure to add ! if this involve a breaking change

🛠️ Fixes #488

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[duwenxin99]

Hi @vijaybalebail, could you sign the CLA here? Thanks!

[duwenxin99]

/gcbrun

[duwenxin99]

@vijaybalebail Thanks for updating. We will request an CLA override before we merge this PR in. One question: can we use https://github.com/sijms/go-ora instead of https://github.com/godror/godror so that the developers don't have to install the OIC?

[duwenxin99]

Hi @vijaybalebail, I'm requesting and license to use Oracle for our internal project which may take some time. I'll update the test config for the PR once it's ready.

[duwenxin99]

Hi @vijaybalebail, I have set up an Oracle 23ai test instance and added the test configs. The environment variables available to use are ORACLE_USER, ORACLE_PASS, and ORACLE_HOST(without port). The guide to integration test is here. Feel free to refer to other sources' test as examples. Let me know if you encounter any issues.

[Matthieu68857]

Is there any plan to add an "oracle-sql" tool that would allow to define tools like other engines?
I tested a build with this PR, and having "just" the "oracle-execute-sql" is limiting.

[duwenxin99]

Is there any plan to add an "oracle-sql" tool that would allow to define tools like other engines? I tested a build with this PR, and having "just" the "oracle-execute-sql" is limiting.

Yes we will support that.

[vijaybalebail]

Oracle Has no lisence for Oracle Clients.

On 9/29/2025 9:21 AM, Wenxin Du wrote: *duwenxin99* left a comment (googleapis/genai-toolbox#1456) <https://urldefense.com/v3/__https://github.com/googleapis/genai-toolbox/pull/1456*issuecomment-3347260043__;Iw!!ACWV5N9M2RV99hQ!JFtO91wE4v4cbPGbCugn5JGyMOWDPcB7Wvi011L3jyHHfJLCSxFISs2W5X9Osfo_Be5Ntf3Kh_6sGbn9U2UGU7nv5JAEXQ$> Hi @vijaybalebail <https://urldefense.com/v3/__https://github.com/vijaybalebail__;!!ACWV5N9M2RV99hQ!JFtO91wE4v4cbPGbCugn5JGyMOWDPcB7Wvi011L3jyHHfJLCSxFISs2W5X9Osfo_Be5Ntf3Kh_6sGbn9U2UGU7m3jyJ4kg$>, I'm requesting and license to use Oracle for our internal project which may take some time. I'll update the test config for the PR once it's ready. — Reply to this email directly, view it on GitHub <https://urldefense.com/v3/__https://github.com/googleapis/genai-toolbox/pull/1456*issuecomment-3347260043__;Iw!!ACWV5N9M2RV99hQ!JFtO91wE4v4cbPGbCugn5JGyMOWDPcB7Wvi011L3jyHHfJLCSxFISs2W5X9Osfo_Be5Ntf3Kh_6sGbn9U2UGU7nv5JAEXQ$>, or unsubscribe <https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/APA2DDLDTUHHE4ZYSNRTFUD3VE54ZAVCNFSM6AAAAACGSHDS4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGNBXGI3DAMBUGM__;!!ACWV5N9M2RV99hQ!JFtO91wE4v4cbPGbCugn5JGyMOWDPcB7Wvi011L3jyHHfJLCSxFISs2W5X9Osfo_Be5Ntf3Kh_6sGbn9U2UGU7knI2kTOw$>. You are receiving this because you were mentioned.Message ID: ***@***.***>

--------------0WdX6tIIMnU0Wax7YHhTqpAr Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 8bit <!DOCTYPE html><html><head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> </head> <body> <p>Oracle Has no lisence for Oracle Clients.</p> <div class="moz-cite-prefix">On 9/29/2025 9:21 AM, Wenxin Du wrote:<br> </div> <blockquote type="cite" ***@***.***"> <div style="display: flex; flex-wrap: wrap; white-space: pre-wrap; align-items: center; "><img height="20" width="20" style="border-radius:50%; margin-right: 4px;" decoding="async" src="https://avatars.githubusercontent.com/u/117315983?s=20&amp;v=4" moz-do-not-send="true"><strong>duwenxin99</strong> left a comment <a href="https://urldefense.com/v3/__https://github.com/googleapis/genai-toolbox/pull/1456*issuecomment-3347260043__;Iw!!ACWV5N9M2RV99hQ!JFtO91wE4v4cbPGbCugn5JGyMOWDPcB7Wvi011L3jyHHfJLCSxFISs2W5X9Osfo_Be5Ntf3Kh_6sGbn9U2UGU7nv5JAEXQ$" moz-do-not-send="true">(googleapis/genai-toolbox#1456)</a></div> <p dir="auto">Hi <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/vijaybalebail/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://urldefense.com/v3/__https://github.com/vijaybalebail__;!!ACWV5N9M2RV99hQ!JFtO91wE4v4cbPGbCugn5JGyMOWDPcB7Wvi011L3jyHHfJLCSxFISs2W5X9Osfo_Be5Ntf3Kh_6sGbn9U2UGU7m3jyJ4kg$" ***@***.***</a>, I'm requesting and license to use Oracle for our internal project which may take some time. I'll update the test config for the PR once it's ready.</p> <p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br> Reply to this email directly, <a href="https://urldefense.com/v3/__https://github.com/googleapis/genai-toolbox/pull/1456*issuecomment-3347260043__;Iw!!ACWV5N9M2RV99hQ!JFtO91wE4v4cbPGbCugn5JGyMOWDPcB7Wvi011L3jyHHfJLCSxFISs2W5X9Osfo_Be5Ntf3Kh_6sGbn9U2UGU7nv5JAEXQ$" moz-do-not-send="true">view it on GitHub</a>, or <a href="https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/APA2DDLDTUHHE4ZYSNRTFUD3VE54ZAVCNFSM6AAAAACGSHDS4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGNBXGI3DAMBUGM__;!!ACWV5N9M2RV99hQ!JFtO91wE4v4cbPGbCugn5JGyMOWDPcB7Wvi011L3jyHHfJLCSxFISs2W5X9Osfo_Be5Ntf3Kh_6sGbn9U2UGU7knI2kTOw$" moz-do-not-send="true">unsubscribe</a>.<br> You are receiving this because you were mentioned.<img src="https://github.com/notifications/beacon/APA2DDNC7BDGPQ5LSU3CT5T3VE54ZA5CNFSM6AAAAACGSHDS4GWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTWHQMRIW.gif" height="1" width="1" alt="" moz-do-not-send="true"><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span>&lt;googleapis/genai-toolbox/pull/1456/c3347260043</span><span>@</span><span>github</span><span>.</span><span>com&gt;</span></span></p> <script type="application/ld+json">[ { ***@***.***": "http://schema.org", ***@***.***": "EmailMessage", "potentialAction": { ***@***.***": "ViewAction", "target": "#1456 (comment)", "url": "#1456 (comment)", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { ***@***.***": "Organization", "name": "GitHub", "url": "https://github.com" } } ]</script> </blockquote> </body> </html>

…

--------------0WdX6tIIMnU0Wax7YHhTqpAr--

[vijaybalebail]

I did add oracle-sql and oracle-execute.

On 10/2/2025 4:14 PM, Wenxin Du wrote: *duwenxin99* left a comment (googleapis/genai-toolbox#1456) <https://urldefense.com/v3/__https://github.com/googleapis/genai-toolbox/pull/1456*issuecomment-3363083980__;Iw!!ACWV5N9M2RV99hQ!NqAMVqHTs7m14eeKTbYEusX4XYpFYcljcSVIUsPLnlck3gxTc6lP-epoNQb-Xq0fLectTgfEODwo5hLgidXVKpcc4ggk9A$> Is there any plan to add an "oracle-sql" tool that would allow to define tools like other engines? I tested a build with this PR, and having "just" the "oracle-execute-sql" is limiting. Yes we will support that. — Reply to this email directly, view it on GitHub <https://urldefense.com/v3/__https://github.com/googleapis/genai-toolbox/pull/1456*issuecomment-3363083980__;Iw!!ACWV5N9M2RV99hQ!NqAMVqHTs7m14eeKTbYEusX4XYpFYcljcSVIUsPLnlck3gxTc6lP-epoNQb-Xq0fLectTgfEODwo5hLgidXVKpcc4ggk9A$>, or unsubscribe <https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/APA2DDNU3OFLPU7KCBRXFYD3VWISRAVCNFSM6AAAAACGSHDS4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGNRTGA4DGOJYGA__;!!ACWV5N9M2RV99hQ!NqAMVqHTs7m14eeKTbYEusX4XYpFYcljcSVIUsPLnlck3gxTc6lP-epoNQb-Xq0fLectTgfEODwo5hLgidXVKpc60WWlSw$>. You are receiving this because you were mentioned.Message ID: ***@***.***>

--------------IQ58wW0sYpzlwuVeP000YbpB Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 8bit <!DOCTYPE html><html><head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> </head> <body> <p>I did add oracle-sql and oracle-execute.</p> <div class="moz-cite-prefix">On 10/2/2025 4:14 PM, Wenxin Du wrote:<br> </div> <blockquote type="cite" ***@***.***"> <div style="display: flex; flex-wrap: wrap; white-space: pre-wrap; align-items: center; "><img height="20" width="20" style="border-radius:50%; margin-right: 4px;" decoding="async" src="https://avatars.githubusercontent.com/u/117315983?s=20&amp;v=4" moz-do-not-send="true"><strong>duwenxin99</strong> left a comment <a href="https://urldefense.com/v3/__https://github.com/googleapis/genai-toolbox/pull/1456*issuecomment-3363083980__;Iw!!ACWV5N9M2RV99hQ!NqAMVqHTs7m14eeKTbYEusX4XYpFYcljcSVIUsPLnlck3gxTc6lP-epoNQb-Xq0fLectTgfEODwo5hLgidXVKpcc4ggk9A$" moz-do-not-send="true">(googleapis/genai-toolbox#1456)</a></div> <blockquote> <p dir="auto">Is there any plan to add an &quot;oracle-sql&quot; tool that would allow to define tools like other engines? I tested a build with this PR, and having &quot;just&quot; the &quot;oracle-execute-sql&quot; is limiting.</p> </blockquote> <p dir="auto">Yes we will support that.</p> <p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br> Reply to this email directly, <a href="https://urldefense.com/v3/__https://github.com/googleapis/genai-toolbox/pull/1456*issuecomment-3363083980__;Iw!!ACWV5N9M2RV99hQ!NqAMVqHTs7m14eeKTbYEusX4XYpFYcljcSVIUsPLnlck3gxTc6lP-epoNQb-Xq0fLectTgfEODwo5hLgidXVKpcc4ggk9A$" moz-do-not-send="true">view it on GitHub</a>, or <a href="https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/APA2DDNU3OFLPU7KCBRXFYD3VWISRAVCNFSM6AAAAACGSHDS4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGNRTGA4DGOJYGA__;!!ACWV5N9M2RV99hQ!NqAMVqHTs7m14eeKTbYEusX4XYpFYcljcSVIUsPLnlck3gxTc6lP-epoNQb-Xq0fLectTgfEODwo5hLgidXVKpc60WWlSw$" moz-do-not-send="true">unsubscribe</a>.<br> You are receiving this because you were mentioned.<img src="https://github.com/notifications/beacon/APA2DDPH7HOKYALDWYC6YST3VWISRA5CNFSM6AAAAACGSHDS4GWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTWIOSLMY.gif" height="1" width="1" alt="" moz-do-not-send="true"><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span>&lt;googleapis/genai-toolbox/pull/1456/c3363083980</span><span>@</span><span>github</span><span>.</span><span>com&gt;</span></span></p> <script type="application/ld+json">[ { ***@***.***": "http://schema.org", ***@***.***": "EmailMessage", "potentialAction": { ***@***.***": "ViewAction", "target": "#1456 (comment)", "url": "#1456 (comment)", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { ***@***.***": "Organization", "name": "GitHub", "url": "https://github.com" } } ]</script> </blockquote> </body> </html>

…

--------------IQ58wW0sYpzlwuVeP000YbpB--

[Matthieu68857]

@vijaybalebail I've tried to build the toolbox with the new tool and I ran into

# github.com/googleapis/genai-toolbox/internal/tools/oracle/oracleexecutesql
  internal/tools/oracle/oracleexecutesql/oracleexecutesql.go:89:30: multiple-value parameters.McpManifest() (value of type (tools.McpToolsSchema, map[string][]string)) in single-value context
  # github.com/googleapis/genai-toolbox/internal/tools/oracle/oraclesql
  internal/tools/oracle/oraclesql/oraclesql.go:84:64: assignment mismatch: 4 variables but tools.ProcessParameters returns 3 values

So I fixed it locally to make it work by updating the manifests of both tools (oracle-sql and oracle-execute-sql).

diff --git a/internal/tools/oracle/oraclesql/oraclesql.go b/internal/tools/oracle/oraclesql/oraclesql.go
index 9fec368..ff1d110 100644
--- a/internal/tools/oracle/oraclesql/oraclesql.go
+++ b/internal/tools/oracle/oraclesql/oraclesql.go
@@ -81,16 +81,12 @@ func (cfg Config) Initialize(srcs map[string]sources.Source) (tools.Tool, error)
                 return nil, fmt.Errorf("invalid source for %q tool: source kind must be one of %q", kind, compatibleSources)
         }

-        allParameters, paramManifest, paramMcpManifest, err := tools.ProcessParameters(cfg.TemplateParameters, cfg.Parameters)
+        allParameters, paramManifest, err := tools.ProcessParameters(cfg.TemplateParameters, cfg.Parameters)
         if err != nil {
                 return nil, fmt.Errorf("error processing parameters: %w", err)
         }

-        mcpManifest := tools.McpManifest{
-                Name:        cfg.Name,
-                Description: cfg.Description,
-                InputSchema: paramMcpManifest,
-        }
+        mcpManifest := tools.GetMcpManifest(cfg.Name, cfg.Description, nil, allParameters)

         // finish tool setup
         t := Tool{

diff --git a/internal/tools/oracle/oracleexecutesql/oracleexecutesql.go b/internal/tools/oracle/oracleexecutesql/oracleexecutesql.go
index f16fbde..7aadb48 100644
--- a/internal/tools/oracle/oracleexecutesql/oracleexecutesql.go
+++ b/internal/tools/oracle/oracleexecutesql/oracleexecutesql.go
@@ -83,11 +83,7 @@ func (cfg Config) Initialize(srcs map[string]sources.Source) (tools.Tool, error)
         sqlParameter := tools.NewStringParameter("sql", "The SQL to execute.")
         parameters := tools.Parameters{sqlParameter}

-        mcpManifest := tools.McpManifest{
-                Name:        cfg.Name,
-                Description: cfg.Description,
-                InputSchema: parameters.McpManifest(),
-        }
+        mcpManifest := tools.GetMcpManifest(cfg.Name, cfg.Description, nil, parameters)

         // finish tool setup
         t := Tool{

As this is a quick fix and not sure I'm legitimate to provide full update, I'll let you adapt if that works for you!