d365fo-mcp 1.5.2 → 1.7.0

package/README.md

@@ -10,7 +10,7 @@
 [![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)
 <!-- 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-72.7%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-91%25-lightgrey.svg)](eval/COVERAGE.md)
 <!-- coverage-badge:end -->
 
 *Grounded AI development for Dynamics 365 Finance & Operations — works with GitHub Copilot and Claude Code*

package/bridge/D365MetadataBridge/D365MetadataBridge.csproj

@@ -23,10 +23,18 @@
 
   <!--
     D365FO bin directory — contains all Microsoft.Dynamics.*.dll assemblies.
-    Every D365FO development desktop has this directory available.
+    Every D365FO development desktop has this directory available, but WHICH
+    volume it sits on is decided by the VM image: K: on cloud-hosted
+    environments, C: on the downloadable VHD, J: on newer images (#769), and
+    nothing stops the next image from using another letter again.
+
+    So the drive is swept rather than guessed: the first
+    <drive>:\AosService\PackagesLocalDirectory\bin that exists wins, with the
+    historically standard letters tried first and the rest of the alphabet
+    after them. MSBuild has no loop at evaluation time, and the property has to
+    be resolved before Reference items are globbed — hence one line per letter.
+    A: and B: are left out on purpose (floppy letters stall the probe).
 
-    Traditional VM:  K:\AosService\PackagesLocalDirectory\bin
-    Cloud-hosted:    C:\AosService\PackagesLocalDirectory\bin
     UDE (Unified Developer Experience): path from .mcp.json microsoftPackagesPath/bin
 
     Override per-machine via:
@@ -36,6 +44,28 @@
   <PropertyGroup>
     <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('K:\AosService\PackagesLocalDirectory\bin')">K:\AosService\PackagesLocalDirectory\bin</D365BinPath>
     <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('C:\AosService\PackagesLocalDirectory\bin')">C:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('J:\AosService\PackagesLocalDirectory\bin')">J:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('I:\AosService\PackagesLocalDirectory\bin')">I:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('D:\AosService\PackagesLocalDirectory\bin')">D:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('E:\AosService\PackagesLocalDirectory\bin')">E:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('F:\AosService\PackagesLocalDirectory\bin')">F:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('G:\AosService\PackagesLocalDirectory\bin')">G:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('H:\AosService\PackagesLocalDirectory\bin')">H:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('L:\AosService\PackagesLocalDirectory\bin')">L:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('M:\AosService\PackagesLocalDirectory\bin')">M:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('N:\AosService\PackagesLocalDirectory\bin')">N:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('O:\AosService\PackagesLocalDirectory\bin')">O:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('P:\AosService\PackagesLocalDirectory\bin')">P:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('Q:\AosService\PackagesLocalDirectory\bin')">Q:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('R:\AosService\PackagesLocalDirectory\bin')">R:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('S:\AosService\PackagesLocalDirectory\bin')">S:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('T:\AosService\PackagesLocalDirectory\bin')">T:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('U:\AosService\PackagesLocalDirectory\bin')">U:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('V:\AosService\PackagesLocalDirectory\bin')">V:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('W:\AosService\PackagesLocalDirectory\bin')">W:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('X:\AosService\PackagesLocalDirectory\bin')">X:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('Y:\AosService\PackagesLocalDirectory\bin')">Y:\AosService\PackagesLocalDirectory\bin</D365BinPath>
+    <D365BinPath Condition="'$(D365BinPath)' == '' And Exists('Z:\AosService\PackagesLocalDirectory\bin')">Z:\AosService\PackagesLocalDirectory\bin</D365BinPath>
   </PropertyGroup>
 
   <!-- NuGet packages -->

package/bridge/D365MetadataBridge/Program.cs

@@ -416,7 +416,7 @@ Usage:
   D365MetadataBridge.exe [options]
 
 Options:
-  --packages-path <path>            Path to primary PackagesLocalDirectory (default: K:\AosService\PackagesLocalDirectory)
+  --packages-path <path>            Path to primary PackagesLocalDirectory (required; the server detects it per machine)
   --reference-packages-path <path>  UDE: secondary packages path (Microsoft FrameworkDirectory). Objects not found in
                                     the primary path are looked up here, enabling resolution of both custom and
                                     Microsoft-shipped metadata in UDE environments.

package/bridge/D365MetadataBridge/Services/MetadataWriteService.cs

@@ -1504,7 +1504,7 @@ namespace D365MetadataBridge.Services
                         ?? throw new ArgumentException($"Table '{objectName}' not found");
                     var msi = GetModelSaveInfoForObject(_provider.Tables, objectName);
                     if (!SetAxTableProperty(obj, propertyPath, propertyValue))
-                        throw new ArgumentException($"Unknown AxTable property '{propertyPath}' — nothing was written. Supported: label, developerDocumentation, configurationKey, formRef, tableGroup, cacheLookup, clusteredIndex, primaryIndex, replacementKey, saveDataPerCompany, tableType, supportInheritance, instanceRelationType, extends, titleField1, titleField2.");
+                        throw new ArgumentException($"Unknown AxTable property '{propertyPath}' — nothing was written. Supported: label, developerDocumentation, configurationKey, formRef, tableGroup, cacheLookup, clusteredIndex, primaryIndex, replacementKey, saveDataPerCompany, allowRowVersionChangeTracking, createdBy, createdDateTime, createdTransactionId, modifiedBy, modifiedDateTime, modifiedTransactionId, tableType, supportInheritance, instanceRelationType, extends, titleField1, titleField2.");
                     ((IMetaTableProvider)_provider.Tables).Update(obj, msi);
                     return new { success = true, operation = "modify-property", objectType, objectName, propertyPath, propertyValue, api = "Update" };
                 }
@@ -1548,6 +1548,16 @@ namespace D365MetadataBridge.Services
                     ((IMetaViewProvider)_provider.Views).Update(obj, msi);
                     return new { success = true, operation = "modify-property", objectType, objectName, propertyPath, propertyValue, api = "Update" };
                 }
+                case "data-entity":
+                {
+                    var obj = _provider.DataEntityViews.Read(objectName)
+                        ?? throw new ArgumentException($"Data entity '{objectName}' not found");
+                    var msi = GetModelSaveInfoForObject(_provider.DataEntityViews, objectName);
+                    if (!SetAxDataEntityViewProperty(obj, propertyPath, propertyValue))
+                        throw new ArgumentException($"Unknown AxDataEntityView property '{propertyPath}' — nothing was written. Supported: label, developerDocumentation, primaryKey, isPublic, publicEntityName, publicCollectionName, dataManagementEnabled, dataManagementStagingTable, entityCategory, allowRowVersionChangeTracking, allowRetention.");
+                    ((IMetaDataEntityViewProvider)_provider.DataEntityViews).Update(obj, msi);
+                    return new { success = true, operation = "modify-property", objectType, objectName, propertyPath, propertyValue, api = "Update" };
+                }
                 case "menu-item-action":
                 {
                     var obj = _provider.MenuItemActions.Read(objectName)
@@ -2642,6 +2652,16 @@ namespace D365MetadataBridge.Services
                 case "savedatapercompany":
                     tbl.SaveDataPerCompany = ParseNoYes(value);
                     break;
+                // Dual-write's table-side change-tracking prerequisite.
+                case "allowrowversionchangetracking":
+                    tbl.AllowRowVersionChangeTracking = ParseNoYes(value);
+                    break;
+                case "createdby": tbl.CreatedBy = ParseNoYes(value); break;
+                case "createddatetime": tbl.CreatedDateTime = ParseNoYes(value); break;
+                case "createdtransactionid": tbl.CreatedTransactionId = ParseNoYes(value); break;
+                case "modifiedby": tbl.ModifiedBy = ParseNoYes(value); break;
+                case "modifieddatetime": tbl.ModifiedDateTime = ParseNoYes(value); break;
+                case "modifiedtransactionid": tbl.ModifiedTransactionId = ParseNoYes(value); break;
                 case "tabletype":
                     if (!Enum.TryParse<Microsoft.Dynamics.AX.Metadata.Core.MetaModel.TableType>(value, true, out var tt))
                         throw new ArgumentException(
@@ -2725,6 +2745,39 @@ namespace D365MetadataBridge.Services
             return true;
         }
 
+        /// <summary>
+        /// AxDataEntityView property setter. AllowRowVersionChangeTracking is
+        /// dual-write's change-tracking switch and is also required on every
+        /// source table.
+        /// </summary>
+        private bool SetAxDataEntityViewProperty(AxDataEntityView e, string prop, string value)
+        {
+            switch (prop.ToLowerInvariant())
+            {
+                case "label": e.Label = value; break;
+                case "developerdocumentation": e.DeveloperDocumentation = value; break;
+                case "primarykey": e.PrimaryKey = value; break;
+                case "publicentityname": e.PublicEntityName = value; break;
+                case "publiccollectionname": e.PublicCollectionName = value; break;
+                case "datamanagementstagingtable": e.DataManagementStagingTable = value; break;
+                case "ispublic": e.IsPublic = ParseNoYes(value); break;
+                case "datamanagementenabled": e.DataManagementEnabled = ParseNoYes(value); break;
+                case "allowrowversionchangetracking": e.AllowRowVersionChangeTracking = ParseNoYes(value); break;
+                case "allowretention": e.AllowRetention = ParseNoYes(value); break;
+                case "entitycategory":
+                    if (!Enum.TryParse<Microsoft.Dynamics.AX.Metadata.Core.MetaModel.EntityCategory>(value, true, out var ec))
+                        throw new ArgumentException(
+                            $"'{value}' is not a valid EntityCategory. Valid values: " +
+                            string.Join(", ", Enum.GetNames(typeof(Microsoft.Dynamics.AX.Metadata.Core.MetaModel.EntityCategory))) + ".");
+                    e.EntityCategory = ec;
+                    break;
+                default:
+                    Console.Error.WriteLine($"[WriteService] Unknown AxDataEntityView property: {prop}");
+                    return false;
+            }
+            return true;
+        }
+
         private bool SetAxViewProperty(AxView v, string prop, string value)
         {
             switch (prop.ToLowerInvariant())

package/dist/bridge/bridgeAdapter.d.ts

@@ -358,7 +358,7 @@ export declare function bridgeDiscoverFormPatterns(bridge: BridgeClient | undefi
 export declare function tryBridgeSecurityArtifact(bridge: BridgeClient | undefined, name: string, artifactType: 'privilege' | 'duty' | 'role', includeChain: boolean): Promise<ToolResult | null>;
 export declare function tryBridgeMenuItem(bridge: BridgeClient | undefined, name: string, itemType?: string): Promise<ToolResult | null>;
 export declare function tryBridgeTableExtensions(bridge: BridgeClient | undefined, baseTableName: string): Promise<ToolResult | null>;
-export declare function tryBridgeCompletion(bridge: BridgeClient | undefined, symbolName: string, prefix?: string): Promise<ToolResult | null>;
+export declare function tryBridgeCompletion(bridge: BridgeClient | undefined, symbolName: string, prefix?: string, ancestors?: string[]): Promise<ToolResult | null>;
 export declare function tryBridgeCocExtensions(bridge: BridgeClient | undefined, baseClassName: string, methodName?: string): Promise<ToolResult | null>;
 export declare function tryBridgeEventHandlers(bridge: BridgeClient | undefined, targetName: string, eventName?: string, handlerType?: string): Promise<ToolResult | null>;
 export declare function tryBridgeApiUsageCallers(bridge: BridgeClient | undefined, apiName: string, limit?: number): Promise<ToolResult | null>;

package/dist/bridge/bridgeAdapter.js

@@ -1077,7 +1077,7 @@ const BRIDGE_MODIFY_OPS = new Set([
  */
 const BRIDGE_MODIFY_TYPES = new Set([
     'class', 'table', 'enum', 'edt',
-    'form', 'query', 'view',
+    'form', 'query', 'view', 'data-entity',
     'class-extension', 'table-extension', 'form-extension', 'enum-extension',
     'menu-item-action', 'menu-item-display', 'menu-item-output',
     'menu',
@@ -1911,13 +1911,38 @@ function formatTableExtensions(r) {
     return out;
 }
 // CODE COMPLETION (Phase 6)
-export async function tryBridgeCompletion(bridge, symbolName, prefix) {
+export async function tryBridgeCompletion(bridge, symbolName, prefix, ancestors) {
     if (!bridge?.isReady || !bridge.metadataAvailable)
         return null;
     try {
         const result = await bridge.getCompletionMembers(symbolName);
         if (!result || !result.members || result.members.length === 0)
             return null;
+        // IMetadataProvider returns DECLARED members only, so a subclass lists
+        // nothing it inherits and the reader concludes the member does not exist.
+        // Merge the base classes in, nearest first; a name already present wins,
+        // since that is the subclass's own override.
+        if (ancestors?.length) {
+            const seen = new Set(result.members.map(m => m.name.toLowerCase()));
+            for (const ancestor of ancestors) {
+                let inherited = null;
+                try {
+                    inherited = await bridge.getCompletionMembers(ancestor);
+                }
+                catch (e) {
+                    // One unreadable link must not discard the members already merged.
+                    console.error(`[BridgeAdapter] getCompletionMembers(${ancestor}) failed: ${e}`);
+                    continue;
+                }
+                for (const m of inherited?.members ?? []) {
+                    const key = m.name.toLowerCase();
+                    if (seen.has(key))
+                        continue;
+                    seen.add(key);
+                    result.members.push({ ...m, inheritedFrom: inherited?.symbolName || ancestor });
+                }
+            }
+        }
         return { content: [{ type: 'text', text: formatCompletion(result, prefix) }] };
     }
     catch (e) {
@@ -1943,9 +1968,15 @@ function formatCompletion(r, prefix) {
     const methodMembers = members.filter(m => m.kind === 'method');
     const fieldMembers = members.filter(m => m.kind === 'field');
     if (methodMembers.length > 0) {
+        const inheritedCount = methodMembers.filter(m => m.inheritedFrom).length;
         out += `## Methods (${methodMembers.length})\n`;
         for (const m of methodMembers) {
-            out += m.signature ? `- \`${m.signature}\`\n` : `- ${m.name}\n`;
+            const body = m.signature ? `\`${m.signature}\`` : m.name;
+            out += m.inheritedFrom ? `- ${body} _(inherited from ${m.inheritedFrom})_\n` : `- ${body}\n`;
+        }
+        if (inheritedCount > 0) {
+            out += `\n> ${inheritedCount} of these are inherited — callable on ${r.symbolName}, but ` +
+                `declared on a base class. To read or wrap one, target the class named beside it.\n`;
         }
     }
     if (fieldMembers.length > 0) {

package/dist/bridge/bridgeClient.d.ts

@@ -19,7 +19,7 @@ export * from './bridgeTypes.js';
 export interface BridgeClientOptions {
     /** Path to the D365MetadataBridge.exe (auto-detected if omitted) */
     bridgeExePath?: string;
-    /** K:\AosService\PackagesLocalDirectory */
+    /** e.g. K:\AosService\PackagesLocalDirectory — the volume varies by VM image */
     packagesPath: string;
     /**
      * Optional secondary packages path.

package/dist/bridge/bridgeClient.js

@@ -17,6 +17,7 @@ import { EventEmitter } from 'events';
 import * as path from 'path';
 import * as fs from 'fs';
 import { fileURLToPath } from 'url';
+import { packagesRoots } from '../utils/packagesRoot.js';
 export * from './bridgeTypes.js';
 const BRIDGE_EXE_NAME = 'D365MetadataBridge.exe';
 /** Parse a positive-integer env var with a fallback (ignores invalid/non-positive values). */
@@ -751,11 +752,8 @@ function detectPackagesPath() {
     const candidates = [
         process.env.D365FO_PACKAGE_PATH ?? '',
         process.env.PACKAGES_PATH ?? '',
-        // Well-known fallback locations (traditional D365FO VM layouts)
-        'C:\\AosService\\PackagesLocalDirectory',
-        'C:\\AOSService\\PackagesLocalDirectory',
-        'J:\\AosService\\PackagesLocalDirectory',
-        'K:\\AosService\\PackagesLocalDirectory',
+        // Whatever AosService volumes this machine actually has (C:, J:, K:, …)
+        ...packagesRoots(),
     ].filter(Boolean);
     for (const p of candidates) {
         // Traditional: bin is directly under packagesPath

package/dist/bridge/bridgeTypes.d.ts

@@ -92,6 +92,14 @@ export interface BridgeMethodInfo {
     returnType?: string;
     source?: string;
     isStatic?: boolean;
+    /**
+     * ⚠️ Never sent by the bridge. The C# MethodInfoModel behind readClass carries
+     * only name/source/isStatic, so this is always undefined on bridge-sourced
+     * methods — it is populated exclusively by the XML parser path
+     * (xmlParser.parseClassFile). Reading it off a readClass result silently
+     * yields nothing; for a bridge-sourced modifier parse the declaration line out
+     * of getCompletionMembers().members[].signature instead.
+     */
     visibility?: string;
 }
 export interface BridgeEnumInfo {
@@ -499,6 +507,13 @@ export interface BridgeCompletionMember {
     name: string;
     signature?: string;
     kind: string;
+    /**
+     * Set by the caller (not the bridge) when the member was picked up from a
+     * base class rather than declared on the requested one. IMetadataProvider
+     * returns declared members only, so inherited members are merged in on the
+     * TypeScript side and tagged here.
+     */
+    inheritedFrom?: string;
 }
 export interface BridgeCompletionResult {
     symbolName: string;

package/dist/cli/commands/doctor.js

@@ -16,6 +16,7 @@ import { checkRelease } from '../npmRegistry.js';
 import { conflictingLegacyValues, readPath, readSetting } from '../settingsStore.js';
 import { instanceTarget, rootTarget } from '../target.js';
 import { isXppConfigStale, listXppConfigs, xppConfigDir } from '../xppConfig.js';
+import { describePackagesRootScan, packagesRoots } from '../../utils/packagesRoot.js';
 const REQUIRED_NODE_MAJOR = 24;
 /**
  * How to reach the wizard from here. `npm run setup` only exists in a checkout
@@ -74,6 +75,40 @@ function checkConfig(target, label) {
         fix: SETUP_COMMAND,
     };
 }
+/**
+ * The configured packages root against what the machine actually has.
+ *
+ * A packagePath pointing at a drive this image does not use is the failure
+ * behind "setup can't find the namespaces" (#769) — the index then builds from
+ * nothing and every lookup comes back empty, with no error saying why. Naming
+ * the detected volume turns that into a one-line fix.
+ */
+function checkPackagesRoot(store, label) {
+    if (!isWindows)
+        return [];
+    const configured = String(readSetting(store, settingByPath('environment.packagePath')) ?? '').trim();
+    const detected = packagesRoots();
+    if (!configured) {
+        // UDE resolves its roots from the XPP config, so silence here is normal.
+        if (detected.length === 0)
+            return [];
+        return [{
+                severity: 'info',
+                message: `${label}: packages root not configured — the server will use ${detected[0]}`,
+            }];
+    }
+    if (fs.existsSync(configured)) {
+        return [{ severity: 'ok', message: `${label}: packages root OK (${configured})` }];
+    }
+    return [{
+            severity: 'fail',
+            message: `${label}: packages root does not exist (${configured})` +
+                (detected.length > 0 ? `\n   found instead: ${detected.join(', ')}` : `\n   ${describePackagesRootScan()}`),
+            fix: detected.length > 0
+                ? `set environment.packagePath to ${detected[0]} (${SETUP_COMMAND})`
+                : `point environment.packagePath at this machine's PackagesLocalDirectory (${SETUP_COMMAND})`,
+        }];
+}
 /** A legacy .env that disagrees with the config is a trap: the config wins. */
 function legacyEnvChecks(target, label) {
     if (!target.envFile || !fs.existsSync(target.store.configPath))
@@ -205,6 +240,8 @@ export async function doctorCommand() {
     emit(checkConfig(root, 'Root'));
     for (const r of legacyEnvChecks(root, 'Root'))
         emit(r);
+    for (const r of checkPackagesRoot(root.store, 'Root'))
+        emit(r);
     // Database (root)
     emit(checkDb(root.store, paths.defaultDb, 'Root'));
     // C# bridge: the only write path; Windows-only.
@@ -246,6 +283,8 @@ export async function doctorCommand() {
                 emit(r);
             for (const r of legacyEnvChecks(target, `Instance '${inst.name}'`))
                 emit(r);
+            for (const r of checkPackagesRoot(target.store, `Instance '${inst.name}'`))
+                emit(r);
             emit(checkDb(target.store, resolve(inst.dir, 'data', 'xpp-metadata.db'), `Instance '${inst.name}'`));
             if (isWindows && isXppConfigStale(target.store)) {
                 emit({

package/dist/cli/commands/instance.js

@@ -8,7 +8,7 @@ import * as fs from 'node:fs';
 import { resolve } from 'node:path';
 import { settingByPath, settingsInSection } from '../../config/settings.js';
 import { pinBridgeExe } from '../bridgePath.js';
-import { finalizeStagedCopilotFiles, maybePrepareCopilotInstructions } from '../copilotFiles.js';
+import { maybePrepareCopilotInstructions } from '../copilotFiles.js';
 import { createInstance, getInstance, listInstances, normalizeInstanceLayout, suggestPort } from '../instances.js';
 import { mcpJsonNote, placementNote, stdioServer } from '../mcpJson.js';
 import { selectXppConfig } from './config.js';
@@ -17,6 +17,7 @@ import { openInstanceStore, readPath, readSetting, saveStore, writeSetting } fro
 import { instanceTarget } from '../target.js';
 import { askConfirm, askSelect, askText, p } from '../ui.js';
 import { listXppConfigs, xppConfigDir } from '../xppConfig.js';
+import { findPackagesRoot } from '../../utils/packagesRoot.js';
 import { rebuildIndex } from './indexCmd.js';
 const dbPathSetting = settingByPath('index.dbPath');
 const xppConfigNameSetting = settingByPath('environment.xppConfigName');
@@ -93,7 +94,10 @@ export async function instanceAddCommand(name, portArg) {
         await selectXppConfig(store);
     }
     else {
-        await askSetting(store, settingByPath('environment.packagePath'), { required: true });
+        await askSetting(store, settingByPath('environment.packagePath'), {
+            required: true,
+            initial: findPackagesRoot() ?? undefined,
+        });
         await askSetting(store, settingByPath('environment.customModels'), { required: true });
     }
     p.log.step('Workspace and naming');
@@ -108,14 +112,13 @@ export async function instanceAddCommand(name, portArg) {
     // Copilot needs the instructions file just as much as it needs .mcp.json,
     // and an instance is somebody's only setup run — asking here is the only
     // chance scenario F gets.
-    const copilotPlan = await maybePrepareCopilotInstructions(String(readSetting(store, settingByPath('workspace.solutionsPath')) ?? ''));
+    await maybePrepareCopilotInstructions(String(readSetting(store, settingByPath('workspace.solutionsPath')) ?? ''));
     // Both ways to reach this instance: the IDE spawning it over stdio with its
     // own config, or an HTTP client on the port it was given.
     mcpJsonNote({
         [`d365fo-${instanceName}`]: stdioServer(store),
         [`d365fo-${instanceName}-http`]: { url: `http://localhost:${port}/mcp/` },
     }, `.mcp.json — keep the stdio entry OR the http one, not both`);
-    finalizeStagedCopilotFiles(copilotPlan);
     placementNote();
     p.note(`Config: ${store.configPath}\n\n` +
         `1. Build the index:  d365fo-mcp instance rebuild ${instanceName}\n` +

package/dist/cli/commands/setup.js

@@ -15,11 +15,12 @@ import { DOTNET_MISSING, dataRoot, installMode, isWindows, paths, repoRoot, setD
 import { settingByPath, settingsInSection } from '../../config/settings.js';
 import { commandExists, runExe, runShell } from '../exec.js';
 import { pinBridgeExe } from '../bridgePath.js';
-import { finalizeStagedCopilotFiles, maybePrepareCopilotInstructions } from '../copilotFiles.js';
+import { maybePrepareCopilotInstructions } from '../copilotFiles.js';
 import { mcpJsonNote, placementNote, stdioServer } from '../mcpJson.js';
 import { checkRelease } from '../npmRegistry.js';
 import { askAdvanced, askSecrets, askSetting, askSettings } from '../settingsPrompt.js';
 import { migrateLegacyEnv, openStore, readSetting, saveStore, writeSetting } from '../settingsStore.js';
+import { findPackagesRoot } from '../../utils/packagesRoot.js';
 import { rootTarget } from '../target.js';
 import { askConfirm, askSelect, askText, p, requireFullInstall } from '../ui.js';
 import { listXppConfigs } from '../xppConfig.js';
@@ -188,7 +189,13 @@ async function configureEnvironment(store, scenario) {
         }
         return 'ude';
     }
-    await askSetting(store, setting('environment.packagePath'), { required: true });
+    // Offer the AosService volume this machine actually has. Which drive that is
+    // differs per VM image (K:, C:, J:, …), and a wrong guess is what turns the
+    // rest of setup into "no namespaces found" (#769).
+    const detected = findPackagesRoot();
+    if (detected)
+        p.log.success(`Found PackagesLocalDirectory at ${detected}`);
+    await askSetting(store, setting('environment.packagePath'), { required: true, initial: detected ?? undefined });
     await askSetting(store, setting('environment.customModels'), { required: true });
     return 'traditional';
 }
@@ -276,7 +283,7 @@ export async function setupCommand() {
         const url = await askText({ message: 'Azure server URL', placeholder: 'https://your-server.azurewebsites.net/mcp/', required: true });
         await configureEnvironment(store, scenario);
         await configureWorkspace(store, scenario);
-        const copilotPlan = await maybePrepareCopilotInstructions(solutionsPath(store));
+        await maybePrepareCopilotInstructions(solutionsPath(store));
         await configureNaming(store);
         await askSecrets(store, ['behavior']);
         await askAdvanced(store, ['environment', 'workspace', 'naming', 'bridge', 'behavior', 'server']);
@@ -286,7 +293,6 @@ export async function setupCommand() {
             'd365fo-azure': { url },
             'd365fo-local': stdioServer(store),
         });
-        finalizeStagedCopilotFiles(copilotPlan);
         placementNote();
         p.outro('Hybrid setup complete — no local index needed (Azure serves the search).');
         return;
@@ -295,7 +301,7 @@ export async function setupCommand() {
     writeSetting(store, setting('server.mode'), 'full');
     await configureEnvironment(store, scenario);
     await configureWorkspace(store, scenario);
-    const copilotPlan = await maybePrepareCopilotInstructions(solutionsPath(store));
+    await maybePrepareCopilotInstructions(solutionsPath(store));
     await configureNaming(store);
     await configureIndex(store);
     let port = Number(readSetting(store, setting('server.port')) ?? 8080);
@@ -313,7 +319,6 @@ export async function setupCommand() {
     }
     if (scenario === 'local-http') {
         mcpJsonNote({ 'd365fo-mcp-tools': { url: `http://localhost:${port}/mcp/` } });
-        finalizeStagedCopilotFiles(copilotPlan);
         placementNote();
         p.outro('Done. Start the server with: d365fo-mcp start');
         return;
@@ -321,7 +326,6 @@ export async function setupCommand() {
     // D / E — the IDE spawns dist/index.js itself and is pointed at the config
     // file; every other setting comes from there.
     mcpJsonNote({ 'd365fo-mcp-tools': stdioServer(store) });
-    finalizeStagedCopilotFiles(copilotPlan);
     placementNote();
     p.outro('Done. VS spawns the server automatically — no manual start needed.');
 }

package/dist/cli/copilotFiles.d.ts

@@ -1,15 +1,9 @@
-export type CopilotFilePlan = {
-    stagingDir?: string;
-};
 /**
  * Offer to place copilot-instructions.md, falling back to a staging folder.
  *
  * `solutionsPath` is what the user gave for workspace.solutionsPath; when it
- * is empty there is nowhere to copy to, so the files are staged instead and
- * the returned plan tells {@link finalizeStagedCopilotFiles} to complete it
- * once the .mcp.json exists.
+ * is empty there is nowhere to copy to, so the file is staged instead and the
+ * README says where it has to end up.
  */
-export declare function maybePrepareCopilotInstructions(solutionsPath: string): Promise<CopilotFilePlan>;
-/** Add the generated .mcp.json to the staging folder — call after mcpJsonNote(). */
-export declare function finalizeStagedCopilotFiles(plan: CopilotFilePlan): void;
+export declare function maybePrepareCopilotInstructions(solutionsPath: string): Promise<void>;
 //# sourceMappingURL=copilotFiles.d.ts.map

package/dist/cli/copilotFiles.js

@@ -6,8 +6,12 @@
  * built-in file tools and edits X++ behind the metadata provider's back. Both
  * files therefore have to be placed for a working install, so the wizards
  * offer to put this one down as well — either straight into the solutions
- * folder, or, when that is unknown, into a staging folder next to the
- * generated .mcp.json with a README naming the two destinations.
+ * folder, or, when that is unknown, into a staging folder with a README
+ * naming the two destinations.
+ *
+ * The README points at the one .mcp.json `mcpJsonNote()` writes in the data
+ * root rather than copying it: the staging folder sits in that same data
+ * root, so a second copy is only one more file to keep in sync.
  *
  * Shared by `setup` (scenarios B–E) and `instance add` (scenario F), which
  * both end on the same two-file placement step.
@@ -28,9 +32,7 @@ function writeCopilotSetupReadme(targetDir, opts) {
         '1. Copy .mcp.json',
         '   - Recommended (all solutions): %USERPROFILE%\\.mcp.json',
         '   - Per solution: place .mcp.json next to your .sln file',
-        opts.includeLocalMcpCopy
-            ? '   - Local copy prepared here: .mcp.json'
-            : `   - Local copy path: ${paths.mcpSuggestion}`,
+        `   - Generated copy: ${paths.mcpSuggestion}`,
         '',
         '2. Copy .github/copilot-instructions.md',
         opts.copilotAlreadyPlaced
@@ -46,16 +48,15 @@ function writeCopilotSetupReadme(targetDir, opts) {
  * Offer to place copilot-instructions.md, falling back to a staging folder.
  *
  * `solutionsPath` is what the user gave for workspace.solutionsPath; when it
- * is empty there is nowhere to copy to, so the files are staged instead and
- * the returned plan tells {@link finalizeStagedCopilotFiles} to complete it
- * once the .mcp.json exists.
+ * is empty there is nowhere to copy to, so the file is staged instead and the
+ * README says where it has to end up.
  */
 export async function maybePrepareCopilotInstructions(solutionsPath) {
     const source = copilotSource();
     if (!fs.existsSync(source)) {
         p.log.warn('Cannot find copilot-instructions.md in this installation; skipping copy helper.\n' +
             `   Looked for: ${source}`);
-        return {};
+        return;
     }
     const wantsDirectCopy = await askConfirm('Create/copy .github/copilot-instructions.md into the solutions folder now?', true);
     const target = solutionsPath.trim();
@@ -63,16 +64,16 @@ export async function maybePrepareCopilotInstructions(solutionsPath) {
         const targetDir = resolve(target, '.github');
         fs.mkdirSync(targetDir, { recursive: true });
         fs.copyFileSync(source, resolve(targetDir, 'copilot-instructions.md'));
-        writeCopilotSetupReadme(target, { includeLocalMcpCopy: false, copilotAlreadyPlaced: true });
+        writeCopilotSetupReadme(target, { copilotAlreadyPlaced: true });
         p.log.success(`Prepared: ${resolve(targetDir, 'copilot-instructions.md')}`);
         p.log.success(`Prepared: ${resolve(target, 'README.md')}`);
-        return {};
+        return;
     }
     const stageDir = resolve(dataRoot(), 'copilot-setup');
     const stageGitHubDir = resolve(stageDir, '.github');
     fs.mkdirSync(stageGitHubDir, { recursive: true });
     fs.copyFileSync(source, resolve(stageGitHubDir, 'copilot-instructions.md'));
-    writeCopilotSetupReadme(stageDir, { includeLocalMcpCopy: false, copilotAlreadyPlaced: false });
+    writeCopilotSetupReadme(stageDir, { copilotAlreadyPlaced: false });
     if (wantsDirectCopy && !target) {
         p.log.warn('Solutions folder is empty, so files were prepared in the local staging folder instead.');
     }
@@ -80,17 +81,5 @@ export async function maybePrepareCopilotInstructions(solutionsPath) {
         p.log.info('Copy was skipped; files were prepared in a local staging folder for later use.');
     }
     p.log.info(`Staging folder: ${stageDir}`);
-    return { stagingDir: stageDir };
-}
-/** Add the generated .mcp.json to the staging folder — call after mcpJsonNote(). */
-export function finalizeStagedCopilotFiles(plan) {
-    if (!plan.stagingDir)
-        return;
-    const localMcp = paths.mcpSuggestion;
-    if (fs.existsSync(localMcp)) {
-        fs.copyFileSync(localMcp, resolve(plan.stagingDir, '.mcp.json'));
-        writeCopilotSetupReadme(plan.stagingDir, { includeLocalMcpCopy: true, copilotAlreadyPlaced: false });
-        p.log.success(`Prepared: ${resolve(plan.stagingDir, '.mcp.json')}`);
-    }
 }
 //# sourceMappingURL=copilotFiles.js.map

package/dist/cli/settingsPrompt.js

@@ -107,9 +107,12 @@ export async function askSetting(store, setting, opts) {
             return list;
         }
         default: {
+            // A stored answer always wins; the caller's suggestion only fills an
+            // empty field, so re-running setup never overwrites a deliberate value
+            // with something merely detected.
             const raw = await askText({
                 message: message(setting, required ? '' : c.dim('(Enter to skip)')),
-                initialValue: initialText(store, setting),
+                initialValue: initialText(store, setting) || opts?.initial || '',
                 placeholder: setting.placeholder,
                 required,
             });

package/dist/config/settings.js

@@ -81,7 +81,9 @@ export const SETTINGS = [
         type: 'path',
         label: 'Packages root (PackagesLocalDirectory)',
         description: 'AOT packages folder (PackagesLocalDirectory) used as the read-only source for indexing. ' +
-            'Machine-wide on a traditional VM; UDE resolves it from the XPP config instead.',
+            'Machine-wide on a traditional VM; UDE resolves it from the XPP config instead. ' +
+            'Left empty, the server scans the machine\'s drives for AosService\\PackagesLocalDirectory — ' +
+            'which volume that is depends on the VM image (K:, C:, J:, …).',
         placeholder: 'C:\\AOSService\\PackagesLocalDirectory',
     },
     {