abap-mcp 0.2.0 → 0.3.1

package/dist/abap.tools.js

@@ -11,6 +11,7 @@ import { ABAP_VERSIONS, runAbaplint } from "./abap/engine.js";
 import { formatAbap } from "./abap/formatter.js";
 import { outlineAbap } from "./abap/outline.js";
 import { checkCloudReadiness } from "./abap/readiness.js";
+import { lookupReleased, RELEASED_API_SNAPSHOT, suggestSuccessor, } from "./abap/released.js";
 import { explainRule, listRules } from "./abap/rules.js";
 import { scaffoldRapBo } from "./abap/scaffold.js";
 import { invalidInput } from "./errors.js";
@@ -132,6 +133,12 @@ export const checkCloudReadinessTool = defineTool({
         brokenAtBaseline: z
             .array(z.unknown())
             .describe("Findings that fail even at the baseline version — fix first, they are not migration items."),
+        releasedApiFindings: z
+            .array(z.unknown())
+            .describe("Released-API observations from the bundled SAP Cloudification snapshot (deprecated-API usage, direct non-released table access with successor hints). Informational — NOT counted in cloudBlockerCount or score."),
+        releasedApiSnapshotDate: z
+            .string()
+            .describe("Date of the bundled released-API snapshot the releasedApiFindings reflect."),
         baselineVersion: z.string().describe("The baseline used."),
         scopeNote: z.string().describe("Exactly what this check does and does not cover."),
     },
@@ -151,6 +158,9 @@ export const checkCloudReadinessTool = defineTool({
             (catLine.length > 0 ? ` [${catLine}]` : "") +
             (report.brokenAtBaseline.length > 0
                 ? `; ${report.brokenAtBaseline.length} finding(s) broken at ${report.baselineVersion} regardless`
+                : "") +
+            (report.releasedApiFindings.length > 0
+                ? `; ${report.releasedApiFindings.length} released-API note(s) (snapshot ${report.releasedApiSnapshotDate})`
                 : "");
         return {
             content: [{ type: "text", text }],
@@ -417,11 +427,114 @@ export const getAbapOutline = defineTool({
         };
     },
 });
+/** Accept either a bare name string or a { name, type? } reference. */
+const objectRefField = z
+    .array(z.union([
+    z.string().describe('A bare object name, e.g. "MARA" or "I_Product".'),
+    z.object({
+        name: z.string().describe('Object name, e.g. "MARA", "I_Product", "BAPI_MATERIAL_GET_DETAIL".'),
+        type: z
+            .string()
+            .optional()
+            .describe('Optional SAP object type to disambiguate same-named objects: "TABL" (table), "CDS_STOB" (CDS view entity), "FUNC" (function module), "CLAS", "INTF", "BDEF". Omit if unsure.'),
+    }),
+]))
+    .min(1)
+    .max(200)
+    .describe('Objects to check, 1–200 per call. Each is a bare name string or a { name, type? } object, e.g. ["MARA", { "name": "I_Product", "type": "CDS_STOB" }].');
+export const checkReleasedApiTool = defineTool({
+    name: "check_released_api",
+    title: "Check ABAP released-API status",
+    description: "Look up ABAP repository objects (DB tables, CDS view entities, function modules, classes, interfaces, …) in " +
+        "SAP's published ABAP Cloudification list and report, per object, whether it is a 'released' API (safe to use in " +
+        "ABAP Cloud / Clean Core), 'deprecated' (released but being retired), or 'not-released' (a classic/internal object " +
+        "that is not a public API — e.g. most classic DDIC tables) — with a curated CDS successor hint for common tables. " +
+        `This reflects SAP's official Cloudification list as bundled in this package (snapshot ${RELEASED_API_SNAPSHOT.snapshotDate}); ` +
+        "it ships offline with the server. " +
+        "Use this when you need to know if your code may reference a given object in ABAP Cloud, or which released CDS view " +
+        "to use instead of a classic table — the released-API half of readiness that check_cloud_readiness deliberately " +
+        "leaves to a system's ATC. " +
+        "It does not connect to any SAP system, does not run ATC, and is only as current as the bundled snapshot — a " +
+        `system's own released-API list (ATC check API_RELEASE_STATE_CHECK / SAP_CP_READINESS) remains authoritative; treat ` +
+        "an 'absent from the list' result as 'not-released as of the snapshot', not as proof. " +
+        'Example: check_released_api({ "objects": ["MARA", "I_Product", "BAPI_MATERIAL_GET_DETAIL"] }).',
+    inputSchema: {
+        objects: objectRefField,
+    },
+    outputSchema: {
+        snapshotDate: z.string().describe("Date of the bundled SAP Cloudification snapshot these results reflect."),
+        source: z.string().describe("URL of the SAP Apache-2.0 source the snapshot was built from."),
+        results: z.array(z.object({
+            name: z.string().describe("The object name as queried."),
+            objectType: z
+                .string()
+                .optional()
+                .describe("SAP object type of the matched record (TABL, CDS_STOB, FUNC, …), if found."),
+            state: z
+                .enum(["released", "deprecated", "not-released"])
+                .describe("'released' = safe public API; 'deprecated' = retiring; 'not-released' = not a public API."),
+            recorded: z
+                .boolean()
+                .describe("true = explicitly present in SAP's snapshot (under the requested type, if one was given); false = absent — 'not released as of the snapshot' by omission only."),
+            applicationComponent: z
+                .string()
+                .optional()
+                .describe("Owning application component of the matched record, if found."),
+            successor: z
+                .string()
+                .optional()
+                .describe("Curated released CDS view-entity successor for a classic table, when one is known."),
+        })),
+    },
+    annotations: { readOnlyHint: true, openWorldHint: false, idempotentHint: true },
+    examples: [
+        {
+            description: "Check a classic table, a released CDS view, and a BAPI in one call.",
+            arguments: { objects: ["MARA", "I_Product", "BAPI_MATERIAL_GET_DETAIL"] },
+        },
+        {
+            description: "Disambiguate a name that exists under more than one object type.",
+            arguments: { objects: [{ name: "I_ProcurementProjectTP", type: "CDS_STOB" }] },
+        },
+    ],
+    handler: (args) => {
+        const results = args.objects.map((ref) => {
+            const name = typeof ref === "string" ? ref : ref.name;
+            const type = typeof ref === "string" ? undefined : ref.type;
+            const hit = lookupReleased(name, type);
+            const successor = suggestSuccessor(name);
+            return {
+                name: hit.name,
+                objectType: hit.objectType,
+                state: hit.state,
+                recorded: hit.recorded,
+                applicationComponent: hit.applicationComponent,
+                ...(successor !== undefined ? { successor } : {}),
+            };
+        });
+        const text = results
+            .map((r) => {
+            const tail = r.successor !== undefined ? ` → use ${r.successor}` : "";
+            const provenance = r.recorded ? "" : " (not in snapshot)";
+            return `${r.name}: ${r.state}${r.objectType !== undefined ? ` (${r.objectType})` : ""}${provenance}${tail}`;
+        })
+            .join("\n");
+        return {
+            content: [{ type: "text", text: `Snapshot ${RELEASED_API_SNAPSHOT.snapshotDate}\n${text}` }],
+            structuredContent: {
+                snapshotDate: RELEASED_API_SNAPSHOT.snapshotDate,
+                source: RELEASED_API_SNAPSHOT.source,
+                results,
+            },
+        };
+    },
+});
 /** Every tool this server exposes. (`tools` alias = the registry-export shape @mcp-kit/lint discovers.) */
 export const ALL_TOOLS = [
     lintAbap,
     checkCloudReadinessTool,
     scaffoldRapBoTool,
+    checkReleasedApiTool,
     listAbapRules,
     explainAbapRule,
     formatAbapTool,

package/dist/cli-commands.d.ts

@@ -17,6 +17,7 @@ export declare function cmdReadiness(argv: string[], io: CliIo): number;
 export declare function cmdScaffold(argv: string[], io: CliIo): number;
 export declare function cmdOutline(argv: string[], io: CliIo): number;
 export declare function cmdExplain(argv: string[], io: CliIo): number;
+export declare function cmdReleased(argv: string[], io: CliIo): number;
 export declare function cmdRules(argv: string[], io: CliIo): number;
-export declare const USAGE = "abap-mcp \u2014 SAP ABAP analysis for AI agents (MCP server) and humans (CLI)\n\nUsage:\n  abap-mcp                       start the MCP server on stdio (for AI clients)\n  abap-mcp lint [paths\u2026]         lint files/dirs   [--abap-version v758|Cloud] [--preset style|full|syntax-only] [--json]\n  abap-mcp readiness [paths\u2026]    ABAP Cloud readiness diff   [--baseline v758] [--fail-below N] [--json]\n  abap-mcp scaffold \u2026            generate a RAP managed BO   (--entity --table --key [--fields n:type,\u2026] [--no-draft] [--provided-key] [--out DIR])\n  abap-mcp outline [paths\u2026]      classes/methods/forms structure   [--json]\n  abap-mcp explain <rule>        explain an abaplint rule\n  abap-mcp rules                 list rules   [--query q] [--tag Security]\n\nExit codes: 0 ok \u00B7 1 findings/validation failed \u00B7 2 usage error";
+export declare const USAGE = "abap-mcp \u2014 SAP ABAP analysis for AI agents (MCP server) and humans (CLI)\n\nUsage:\n  abap-mcp                       start the MCP server on stdio (for AI clients)\n  abap-mcp lint [paths\u2026]         lint files/dirs   [--abap-version v758|Cloud] [--preset style|full|syntax-only] [--json]\n  abap-mcp readiness [paths\u2026]    ABAP Cloud readiness diff   [--baseline v758] [--fail-below N] [--json]\n  abap-mcp scaffold \u2026            generate a RAP managed BO   (--entity --table --key [--fields n:type,\u2026] [--no-draft] [--provided-key] [--out DIR])\n  abap-mcp outline [paths\u2026]      classes/methods/forms structure   [--json]\n  abap-mcp released <names\u2026>     released-API status from the bundled SAP snapshot   [--type TABL|FUNC|\u2026] [--json]\n  abap-mcp explain <rule>        explain an abaplint rule\n  abap-mcp rules                 list rules   [--query q] [--tag Security]\n\nExit codes: 0 ok \u00B7 1 findings/validation failed \u00B7 2 usage error";
 export declare function runCli(argv: string[], io: CliIo): number | null;

package/dist/cli-commands.js

@@ -5,22 +5,30 @@
  * story); the *CLI* is a local developer tool, so reading files from disk here
  * is fine. Both call the identical engine — one definition of "clean".
  */
-import { readdirSync, readFileSync, statSync, writeFileSync, mkdirSync, existsSync } from "node:fs";
+import { readdirSync, readFileSync, realpathSync, statSync, writeFileSync, mkdirSync, existsSync } from "node:fs";
 import { basename, extname, join } from "node:path";
 import { ABAP_VERSIONS, MAX_FILES, runAbaplint } from "./abap/engine.js";
 import { outlineAbap } from "./abap/outline.js";
 import { checkCloudReadiness, SCOPE_NOTE } from "./abap/readiness.js";
+import { lookupReleased, RELEASED_API_SNAPSHOT, suggestSuccessor } from "./abap/released.js";
 import { explainRule, listRules } from "./abap/rules.js";
 import { scaffoldRapBo } from "./abap/scaffold.js";
 const ABAP_FILE_RE = /\.(clas\.abap|clas\.locals_imp\.abap|clas\.locals_def\.abap|clas\.testclasses\.abap|prog\.abap|intf\.abap|fugr\.abap|ddls\.asddls|bdef\.asbdef|srvd\.srvdsrv|ddlx\.asddlx)$/;
 /** Recursively collect analyzable sources from file/dir paths. */
 export function collectFiles(paths, io) {
     const found = [];
+    const visitedDirs = new Set();
     const visit = (p) => {
         const st = statSync(p);
         if (st.isDirectory()) {
             if (basename(p) === ".git" || basename(p) === "node_modules")
                 return;
+            // statSync follows symlinks — track real paths so a symlink cycle
+            // can't recurse forever.
+            const real = realpathSync(p);
+            if (visitedDirs.has(real))
+                return;
+            visitedDirs.add(real);
             for (const entry of readdirSync(p))
                 visit(join(p, entry));
             return;
@@ -104,9 +112,13 @@ export function mergeReadiness(reports, baseline) {
     const categories = new Map();
     let blockers = 0;
     const broken = [];
+    const releasedApiFindings = [];
+    let snapshotDate = "";
     for (const r of reports) {
         blockers += r.cloudBlockerCount;
         broken.push(...r.brokenAtBaseline);
+        releasedApiFindings.push(...r.releasedApiFindings);
+        snapshotDate = r.releasedApiSnapshotDate;
         for (const c of r.categories) {
             const cur = categories.get(c.category);
             if (cur === undefined)
@@ -131,6 +143,8 @@ export function mergeReadiness(reports, baseline) {
         cloudBlockerCount: blockers,
         categories: [...categories.values()].sort((a, b) => b.count - a.count),
         brokenAtBaseline: broken,
+        releasedApiFindings,
+        releasedApiSnapshotDate: snapshotDate,
         baselineVersion: baseline,
         scopeNote: SCOPE_NOTE,
     };
@@ -155,6 +169,11 @@ export function cmdReadiness(argv, io) {
             io.out(`  ${c.category.padEnd(18)} ${String(c.count).padStart(4)}  ${c.label}`);
         if (merged.brokenAtBaseline.length > 0)
             io.out(`${merged.brokenAtBaseline.length} finding(s) broken at ${baseline} regardless (fix first; not migration work)`);
+        if (merged.releasedApiFindings.length > 0) {
+            io.out(`${merged.releasedApiFindings.length} released-API note(s) (snapshot ${merged.releasedApiSnapshotDate}; informational, not scored):`);
+            for (const f of merged.releasedApiFindings)
+                io.out(`  ${f.file}:${f.line} [${f.state}] ${f.object}${f.successor !== undefined ? ` → ${f.successor}` : ""}`);
+        }
         io.out(`Note: ${merged.scopeNote}`);
     }
     const failBelow = flags.get("fail-below");
@@ -202,7 +221,13 @@ export function cmdScaffold(argv, io) {
             writeFileSync(target, f.content, "utf8");
             io.out(`wrote ${target}  [${f.validated}]`);
         }
-        writeFileSync(join(outDir, `${sqlTable.toLowerCase()}.tabl.suggestion.txt`), result.suggestedTableDdl, "utf8");
+        const suggestionTarget = join(outDir, `${sqlTable.toLowerCase()}.tabl.suggestion.txt`);
+        if (existsSync(suggestionTarget) && !flags.has("force")) {
+            io.err(`refusing to overwrite ${suggestionTarget} (use --force)`);
+            return 1;
+        }
+        writeFileSync(suggestionTarget, result.suggestedTableDdl, "utf8");
+        io.out(`wrote ${suggestionTarget}`);
     }
     else {
         for (const f of result.files) {
@@ -257,6 +282,30 @@ export function cmdExplain(argv, io) {
     io.out(`${d.key} — ${d.title}\n${d.shortDescription}\n${d.extendedInformation}\nDocs: ${d.docsUrl}`);
     return 0;
 }
+export function cmdReleased(argv, io) {
+    const { flags, rest } = parseFlags(argv);
+    if (rest.length === 0) {
+        io.err("Usage: abap-mcp released <object-name…>   [--type TABL|CDS_STOB|FUNC|…] [--json]");
+        return 2;
+    }
+    const type = typeof flags.get("type") === "string" ? flags.get("type") : undefined;
+    const results = rest.map((name) => {
+        const hit = lookupReleased(name, type);
+        const successor = suggestSuccessor(name);
+        return { ...hit, successor };
+    });
+    if (flags.has("json")) {
+        io.out(JSON.stringify({ snapshotDate: RELEASED_API_SNAPSHOT.snapshotDate, source: RELEASED_API_SNAPSHOT.source, results }, null, 2));
+        return 0;
+    }
+    io.out(`Released-API status (SAP Cloudification snapshot ${RELEASED_API_SNAPSHOT.snapshotDate}):`);
+    for (const r of results) {
+        const tail = r.successor !== undefined ? `  → use ${r.successor}` : "";
+        const provenance = r.recorded ? "" : " (not in snapshot)";
+        io.out(`  ${r.name.padEnd(34)} ${r.state.padEnd(13)} ${(r.objectType ?? "").padEnd(9)}${provenance}${tail}`);
+    }
+    return 0;
+}
 export function cmdRules(argv, io) {
     const { flags } = parseFlags(argv);
     const q = flags.get("query");
@@ -275,6 +324,7 @@ Usage:
   abap-mcp readiness [paths…]    ABAP Cloud readiness diff   [--baseline v758] [--fail-below N] [--json]
   abap-mcp scaffold …            generate a RAP managed BO   (--entity --table --key [--fields n:type,…] [--no-draft] [--provided-key] [--out DIR])
   abap-mcp outline [paths…]      classes/methods/forms structure   [--json]
+  abap-mcp released <names…>     released-API status from the bundled SAP snapshot   [--type TABL|FUNC|…] [--json]
   abap-mcp explain <rule>        explain an abaplint rule
   abap-mcp rules                 list rules   [--query q] [--tag Security]
 
@@ -293,6 +343,8 @@ export function runCli(argv, io) {
             return cmdScaffold(rest, io);
         case "outline":
             return cmdOutline(rest, io);
+        case "released":
+            return cmdReleased(rest, io);
         case "explain":
             return cmdExplain(rest, io);
         case "rules":