d365fo-mcp 1.6.0 → 1.8.0

package/README.md

@@ -7,10 +7,10 @@
 [![npm](https://img.shields.io/npm/v/d365fo-mcp.svg?logo=npm&color=cb3837)](https://www.npmjs.com/package/d365fo-mcp)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Node.js](https://img.shields.io/badge/node-%3E%3D24.0.0-brightgreen.svg)](https://nodejs.org/)
-[![TypeScript](https://img.shields.io/badge/TypeScript-6.0-blue.svg)](https://www.typescriptlang.org/)
-[![Tests](https://img.shields.io/badge/tests-1400%2B-brightgreen.svg)](docs/TESTING.md)
+[![TypeScript](https://img.shields.io/badge/TypeScript-7.0-blue.svg)](https://www.typescriptlang.org/)
+[![Tests](https://img.shields.io/badge/tests-2500%2B-brightgreen.svg)](docs/TESTING.md)
 <!-- coverage-badge:start -->
-[![Core coverage](https://img.shields.io/badge/core_coverage-100%25-brightgreen.svg)](eval/COVERAGE.md) [![Total coverage](https://img.shields.io/badge/total_coverage-85.9%25-lightgrey.svg)](eval/COVERAGE.md)
+[![Core coverage](https://img.shields.io/badge/core_coverage-100%25-brightgreen.svg)](eval/COVERAGE.md) [![Total coverage](https://img.shields.io/badge/total_coverage-100%25-lightgrey.svg)](eval/COVERAGE.md)
 <!-- coverage-badge:end -->
 
 *Grounded AI development for Dynamics 365 Finance & Operations — works with GitHub Copilot and Claude Code*
@@ -51,7 +51,7 @@ This server pre-indexes your entire D365FO installation (580 000+ symbols across
 | 🔍 **Full-codebase intelligence** | 580K+ symbols indexed: classes, tables, forms, EDTs, enums, labels (20M+ rows), security artifacts — FTS5 search in < 10 ms |
 | 🛡️ **Grounded generation** | Fail-closed gates: `prepare` issues grounding tokens, `validate_code(mode="references")` proves every identifier, `validate_code(mode="syntax")` enforces best practices — hallucinated code never reaches disk |
 | 🧩 **Form pattern engine** | Complete catalog of Microsoft form patterns and sub-patterns: recommends the right pattern, clones reference forms with datasource re-binding, **deterministically expands** patterns that have no reference form, **auto-repairs** a form's missing required controls, validates structure and blocks invalid writes |
-| ✍️ **Safe metadata writes** | C# bridge uses Microsoft's own `IMetadataProvider` — no string-replacement XML corruption, automatic `.rnrproj` registration, one-call undo |
+| ✍️ **Safe metadata writes** | C# bridge uses Microsoft's own `IMetadataProvider` wherever it can express the object; the few types and ops it cannot go through structured XML writers with ambiguity guards — never blind string replacement. Automatic `.rnrproj` registration, one-call undo |
 | 🏗️ **SDLC integration** | MSBuild compilation with structured diagnostics, DB sync, xppbp best practices, SysTestRunner — all from chat |
 | 📐 **X++ knowledge base** | Queryable rules: select grammar, CoC authoring, financial dimensions, the posting engine (`LedgerVoucher`), number sequences, `SysExtension`, Electronic Reporting, AX2012→D365FO migration — prevents deprecated APIs |
 
@@ -120,6 +120,7 @@ Deployment guide: [docs/SETUP_AZURE.md](docs/SETUP_AZURE.md) — includes CI/CD
 | [Claude Code setup](docs/SETUP.md#claude-code-cli) | [Configuration](docs/CONFIGURATION.md) | [Testing](docs/TESTING.md) |
 | [Usage examples](docs/USAGE_EXAMPLES.md) — real tool chains | [Architecture](docs/ARCHITECTURE.md) | [Custom / ISV models](docs/CUSTOM_EXTENSIONS.md) |
 | | | [Coverage](eval/COVERAGE.md) — what the badge counts |
+| | | [Eval loop](docs/AGENT_EVAL_LOOP.md) — the self-improvement harness |
 
 ## License
 

package/bridge/D365MetadataBridge/Protocol/RequestDispatcher.cs

@@ -457,11 +457,18 @@ namespace D365MetadataBridge.Protocol
                                 ?? throw new ArgumentException("Missing: objectName");
                             var fieldName = request.GetStringParam("fieldName")
                                 ?? throw new ArgumentException("Missing: fieldName");
+                            // dataField/dataSource select the data-entity-extension mapped-field
+                            // path inside AddField; both are absent for table/table-extension.
+                            // This is the single-op RPC that BridgeClient.addField() calls —
+                            // it must forward the same parameters as the batch-modify case below.
                             return _writeService!.AddField(tableName, fieldName,
                                 request.GetStringParam("fieldType") ?? "String",
                                 request.GetStringParam("edt"),
                                 request.GetBoolParam("mandatory") ?? false,
-                                request.GetStringParam("label"));
+                                request.GetStringParam("label"),
+                                request.GetStringParam("dataField"),
+                                request.GetStringParam("dataSource"),
+                                request.GetStringParam("fieldGroupName"));
                         });
 
                     case "setproperty":
@@ -528,6 +535,53 @@ namespace D365MetadataBridge.Protocol
                             return _writeService!.RemoveIndex(tableName, indexName);
                         });
 
+                    // This is the single-op RPC that BridgeClient.addFullTextIndex()/
+                    // addTableMapping() actually call — it must carry the same four cases
+                    // HandleBatchModify() has below, or every real request dies with
+                    // "-32601 Unknown method" no matter how correct the C# write path is.
+                    case "addfulltextindex":
+                        return HandleWrite(request, () =>
+                        {
+                            var tableName = request.GetStringParam("objectName")
+                                ?? throw new ArgumentException("Missing: objectName");
+                            var indexName = request.GetStringParam("indexName")
+                                ?? throw new ArgumentException("Missing: indexName");
+                            return _writeService!.AddFullTextIndex(tableName, indexName,
+                                request.GetParam<System.Collections.Generic.List<string>>("fields"));
+                        });
+
+                    case "removefulltextindex":
+                        return HandleWrite(request, () =>
+                        {
+                            var tableName = request.GetStringParam("objectName")
+                                ?? throw new ArgumentException("Missing: objectName");
+                            var indexName = request.GetStringParam("indexName")
+                                ?? throw new ArgumentException("Missing: indexName");
+                            return _writeService!.RemoveFullTextIndex(tableName, indexName);
+                        });
+
+                    case "addtablemapping":
+                        return HandleWrite(request, () =>
+                        {
+                            var tableName = request.GetStringParam("objectName")
+                                ?? throw new ArgumentException("Missing: objectName");
+                            var mapName = request.GetStringParam("mapName")
+                                ?? throw new ArgumentException("Missing: mapName");
+                            return _writeService!.AddTableMapping(tableName, mapName,
+                                request.GetStringParam("mappingTable"),
+                                request.GetParam<System.Collections.Generic.List<WriteMappingConnection>>("connections"));
+                        });
+
+                    case "removetablemapping":
+                        return HandleWrite(request, () =>
+                        {
+                            var tableName = request.GetStringParam("objectName")
+                                ?? throw new ArgumentException("Missing: objectName");
+                            var mapName = request.GetStringParam("mapName")
+                                ?? throw new ArgumentException("Missing: mapName");
+                            return _writeService!.RemoveTableMapping(tableName, mapName);
+                        });
+
                     case "addrelation":
                         return HandleWrite(request, () =>
                         {
@@ -588,7 +642,12 @@ namespace D365MetadataBridge.Protocol
                                 ?? throw new ArgumentException("Missing: fieldGroupName");
                             var fieldName = request.GetStringParam("fieldName")
                                 ?? throw new ArgumentException("Missing: fieldName");
-                            return _writeService!.AddFieldToFieldGroup(tableName, groupName, fieldName);
+                            // This is the single-op RPC BridgeClient.addFieldToFieldGroup() actually
+                            // calls — dropping extendBaseFieldGroup here silently reverts every
+                            // caller to the extension-owns-the-group path (see HandleBatchModify()
+                            // below, which forwards it correctly).
+                            return _writeService!.AddFieldToFieldGroup(tableName, groupName, fieldName,
+                                request.GetBoolParam("extendBaseFieldGroup") ?? false);
                         });
 
                     case "modifyfield":
@@ -911,12 +970,17 @@ namespace D365MetadataBridge.Protocol
 
                             case "addfield":
                             case "add-field":
+                                // dataField/dataSource select the data-entity-extension mapped-field
+                                // path inside AddField; both are absent for table/table-extension.
                                 writeResult = _writeService.AddField(objectName,
                                     S("fieldName") ?? throw new ArgumentException("Missing: fieldName"),
                                     S("fieldType") ?? "String",
                                     S("edt"),
                                     B("mandatory") ?? false,
-                                    S("label"));
+                                    S("label"),
+                                    S("dataField"),
+                                    S("dataSource"),
+                                    S("fieldGroupName"));
                                 break;
 
                             case "modifyfield":
@@ -963,6 +1027,33 @@ namespace D365MetadataBridge.Protocol
                                     S("indexName") ?? throw new ArgumentException("Missing: indexName"));
                                 break;
 
+                            case "addfulltextindex":
+                            case "add-full-text-index":
+                                writeResult = _writeService.AddFullTextIndex(objectName,
+                                    S("indexName") ?? throw new ArgumentException("Missing: indexName"),
+                                    op.GetTypedParam<System.Collections.Generic.List<string>>("fields"));
+                                break;
+
+                            case "removefulltextindex":
+                            case "remove-full-text-index":
+                                writeResult = _writeService.RemoveFullTextIndex(objectName,
+                                    S("indexName") ?? throw new ArgumentException("Missing: indexName"));
+                                break;
+
+                            case "addtablemapping":
+                            case "add-table-mapping":
+                                writeResult = _writeService.AddTableMapping(objectName,
+                                    S("mapName") ?? throw new ArgumentException("Missing: mapName"),
+                                    S("mappingTable"),
+                                    op.GetTypedParam<System.Collections.Generic.List<WriteMappingConnection>>("connections"));
+                                break;
+
+                            case "removetablemapping":
+                            case "remove-table-mapping":
+                                writeResult = _writeService.RemoveTableMapping(objectName,
+                                    S("mapName") ?? throw new ArgumentException("Missing: mapName"));
+                                break;
+
                             case "addrelation":
                             case "add-relation":
                                 writeResult = _writeService.AddRelation(objectName,
@@ -998,7 +1089,8 @@ namespace D365MetadataBridge.Protocol
                             case "add-field-to-field-group":
                                 writeResult = _writeService.AddFieldToFieldGroup(objectName,
                                     S("fieldGroupName") ?? S("groupName") ?? throw new ArgumentException("Missing: fieldGroupName"),
-                                    S("fieldName") ?? throw new ArgumentException("Missing: fieldName"));
+                                    S("fieldName") ?? throw new ArgumentException("Missing: fieldName"),
+                                    B("extendBaseFieldGroup") ?? false);
                                 break;
 
                             case "addenumvalue":