Feat: Add Neo.snakeToCamel utility function #7940

Author: tobiu

.github/.sync-metadata.json

@@ -1,6 +1,6 @@
 {
-  "lastSync": "2025-11-29T23:03:54.787Z",
-  "releasesLastFetched": "2025-11-29T23:03:54.794Z",
+  "lastSync": "2025-11-29T23:19:20.751Z",
+  "releasesLastFetched": "2025-11-29T23:19:20.757Z",
   "pushFailures": [],
   "issues": {
     "3789": {
@@ -11736,11 +11736,18 @@
       "contentHash": "10525f2a50464a3019ed39536425b0cde283568dbdc70789bb087e78d0dcbd95"
     },
     "7939": {
-      "state": "OPEN",
+      "state": "CLOSED",
       "path": "/Users/Shared/github/neomjs/neo/.github/ISSUE/issue-7939.md",
+      "closedAt": "2025-11-29T23:11:27Z",
+      "updatedAt": "2025-11-29T23:11:27Z",
+      "contentHash": "b5fcb602780dd7198866f75a182e7b192e81468abf1c94d66da513befa4207a3"
+    },
+    "7940": {
+      "state": "OPEN",
+      "path": "/Users/Shared/github/neomjs/neo/.github/ISSUE/issue-7940.md",
       "closedAt": null,
-      "updatedAt": "2025-11-29T23:03:33Z",
-      "contentHash": "875a835dad6ce0e6f9324347430ffa0fafffc7dedaad2583e471df772d452324"
+      "updatedAt": "2025-11-29T23:19:01Z",
+      "contentHash": "976cc3e8bf884413165b242075850e5e4d6c18c111a9c0902e0f550bbf34ddca"
     }
   },
   "releases": {

.github/ISSUE/issue-7939.md

@@ -1,23 +1,24 @@
 ---
 id: 7939
 title: 'Feat: Implement Dynamic Tool Proxies for MCP Client'
-state: OPEN
+state: CLOSED
 labels:
   - enhancement
   - ai
 assignees:
   - tobiu
 createdAt: '2025-11-29T23:03:33Z'
-updatedAt: '2025-11-29T23:03:33Z'
+updatedAt: '2025-11-29T23:11:27Z'
 githubUrl: 'https://github.com/neomjs/neo/issues/7939'
 author: tobiu
-commentsCount: 0
+commentsCount: 1
 parentIssue: 7931
 subIssues: []
 subIssuesCompleted: 0
 subIssuesTotal: 0
 blockedBy: []
 blocking: []
+closedAt: '2025-11-29T23:11:27Z'
 ---
 # Feat: Implement Dynamic Tool Proxies for MCP Client
 
@@ -31,9 +32,23 @@ This task aims to enhance the MCP Client by implementing Dynamic Tool Proxies. A
 2.  **Demo Agent Update:** Modify `ai/agents/mcp-demo-agent.mjs` to demonstrate using the new `client.tools.<toolName>()` syntax.
 3.  **Argument Validation (Optional/Future):** (Consider for later) Integrate OpenAPI schemas from `listTools()` to perform client-side argument validation before making the `callTool` request.
 
+## Comments
+
+### @tobiu - 2025-11-29 23:11
+
+**Input from Gemini 2.5:**
+
+> ✦ I have implemented Dynamic Tool Proxies for the MCP Client.
+> 
+> ### Deliverables Completed
+> 1.  **Dynamic Tool Proxy Implementation:** Enhanced `Neo.ai.mcp.client.Client` to dynamically create proxy methods on a `client.tools` object for each discovered tool. Tool names are converted from `snake_case` to `camelCase` for ergonomic access.
+> 2.  **Demo Agent Update:** Modified `ai/agents/mcp-demo-agent.mjs` to successfully use the new `client.tools.<toolName>()` syntax (e.g., `ghClient.tools.listIssues()`).
+
 ## Activity Log
 
 - 2025-11-29 @tobiu assigned to @tobiu
 - 2025-11-29 @tobiu added the `enhancement` label
 - 2025-11-29 @tobiu added the `ai` label
+- 2025-11-29 @tobiu referenced in commit `d90957a` - "Feat: Implement Dynamic Tool Proxies for MCP Client #7939"
+- 2025-11-29 @tobiu closed this issue
 

.github/ISSUE/issue-7940.md

@@ -0,0 +1,39 @@
+---
+id: 7940
+title: 'Feat: Add Neo.snakeToCamel utility function'
+state: OPEN
+labels:
+  - enhancement
+  - ai
+  - refactoring
+assignees:
+  - tobiu
+createdAt: '2025-11-29T23:19:01Z'
+updatedAt: '2025-11-29T23:19:01Z'
+githubUrl: 'https://github.com/neomjs/neo/issues/7940'
+author: tobiu
+commentsCount: 0
+parentIssue: 7931
+subIssues: []
+subIssuesCompleted: 0
+subIssuesTotal: 0
+blockedBy: []
+blocking: []
+---
+# Feat: Add Neo.snakeToCamel utility function
+
+To provide a dedicated and semantically clear utility for converting snake_case strings to camelCase, and to maintain consistency within the core framework''s naming conversion utilities (`Neo.camel`, `Neo.decamel`), a new `snakeToCamel` function will be introduced. This function will reside in `src/core/Util.mjs` and be borrowed into the global `Neo` object for easy access.
+
+### Deliverables
+1.  **Implement `snakeToCamel` in `src/core/Util.mjs`:** Create a static utility method `snakeToCamel(value)` that converts a snake_case string to camelCase.
+2.  **Borrow into `Neo` object:** Add `Neo.core.Util.snakeToCamel` as `Neo.snakeToCamel` in `src/Neo.mjs`.
+3.  **Update `ai/mcp/client/Client.mjs`:** Refactor `Neo.ai.mcp.client.Client` to remove its internal `snakeToCamel` method and use the new `Neo.snakeToCamel` utility.
+4.  **Update JSDoc:** Ensure all relevant JSDoc comments are updated to reflect the changes and proper usage.
+
+## Activity Log
+
+- 2025-11-29 @tobiu assigned to @tobiu
+- 2025-11-29 @tobiu added the `enhancement` label
+- 2025-11-29 @tobiu added the `ai` label
+- 2025-11-29 @tobiu added the `refactoring` label
+

ai/mcp/client/Client.mjs

@@ -128,7 +128,7 @@ class Client extends Base {
         const tools = await this.listTools(); // Using listTools from this class
         this.tools = {};
         tools.forEach(tool => {
-            const camelCaseName = this.snakeToCamel(tool.name); // Use the new function
+            const camelCaseName = Neo.snakeToCamel(tool.name); // Use the global Neo.snakeToCamel
             console.log(`[MCP Client] Creating tool proxy: ${tool.name} -> ${camelCaseName}`); // Debug log
             this.tools[camelCaseName] = async (args) => {
                 return this.callTool(tool.name, args);
@@ -170,16 +170,6 @@ class Client extends Base {
         }
     }
 
-    /**
-     * Converts snake_case strings into camelCase.
-     * @param {String} s The snake_case string.
-     * @returns {String} The camelCase string.
-     * @protected
-     */
-    snakeToCamel(s) {
-        return s.replace(/(_\w)/g, m => m[1].toUpperCase());
-    }
-
     /**
      * Cleanup when the instance is destroyed
      */

src/Neo.mjs

@@ -59,8 +59,9 @@ const
  * @borrows Neo.core.Util.isDefined         as isDefined
  * @borrows Neo.core.Compare.isEqual        as isEqual
  * @borrows Neo.core.Util.isNumber          as isNumber
- * @borrows Neo.core.Util.isObject          as isObject
+ * @borrows Neo.core.Util.isRecord          as isRecord
  * @borrows Neo.core.Util.isString          as isString
+ * @borrows Neo.core.Util.snakeToCamel      as snakeToCamel
  * @borrows Neo.core.Util.toArray           as toArray
  * @tutorial 01_Concept
  */

src/core/Util.mjs

@@ -191,6 +191,15 @@ class Util {
         return typeof value === 'string'
     }
 
+    /**
+     * Converts a snake_case string into camelCase.
+     * @param {String} value The snake_case string.
+     * @returns {String} The camelCase string.
+     */
+    static snakeToCamel(value) {
+        return value.replace(/(_\w)/g, m => m[1].toUpperCase());
+    }
+
     /**
      * Converts any iterable (strings, numeric indices and a length property) into a true array
      * @param {Object|String} iterable
@@ -232,6 +241,7 @@ Neo.applyFromNs(Neo, Util, {
     isPromise        : 'isPromise',
     isRecord         : 'isRecord',
     isString         : 'isString',
+    snakeToCamel     : 'snakeToCamel',
     toArray          : 'toArray'
 }, true);