@adhisang/minecraft-modding-mcp 3.2.0 → 4.0.0

package/dist/storage/artifacts-repo.d.ts

@@ -3,6 +3,7 @@ import type { ArtifactProvenance, ArtifactRow, SourceMapping, SourceOrigin } fro
 type SqliteDatabase = InstanceType<typeof Database>;
 interface UpsertArtifactInput {
     artifactId: string;
+    alias?: string;
     origin: SourceOrigin;
     coordinate?: string;
     version?: string;
@@ -27,6 +28,7 @@ export declare class ArtifactsRepo {
     private readonly upsertStmt;
     private readonly getStmt;
     private readonly touchStmt;
+    private readonly setAliasStmt;
     private readonly deleteStmt;
     private readonly listStmt;
     private readonly countStmt;
@@ -34,8 +36,9 @@ export declare class ArtifactsRepo {
     private readonly listLruWithContentBytesStmt;
     constructor(db: SqliteDatabase);
     upsertArtifact(input: UpsertArtifactInput): void;
-    getArtifact(artifactId: string): ArtifactRow | undefined;
+    getArtifact(artifactIdOrAlias: string): ArtifactRow | undefined;
     touchArtifact(artifactId: string, timestamp: string): void;
+    setAlias(artifactId: string, alias: string): void;
     deleteArtifact(artifactId: string): void;
     listArtifactsByLru(limit: number): ArtifactRow[];
     countArtifacts(): number;

package/dist/storage/artifacts-repo.js

@@ -35,6 +35,7 @@ function parseQualityFlags(value) {
 function toArtifactRow(record) {
     return {
         artifactId: record.artifact_id,
+        alias: record.alias ?? undefined,
         origin: record.origin,
         coordinate: record.coordinate ?? undefined,
         version: record.version ?? undefined,
@@ -59,6 +60,7 @@ export class ArtifactsRepo {
     upsertStmt;
     getStmt;
     touchStmt;
+    setAliasStmt;
     deleteStmt;
     listStmt;
     countStmt;
@@ -68,11 +70,12 @@ export class ArtifactsRepo {
         this.db = db;
         this.upsertStmt = this.db.prepare(`
       INSERT INTO artifacts (
-        artifact_id, origin, coordinate, version, binary_jar_path, source_jar_path, repo_url, requested_mapping, mapping_applied, provenance_json, quality_flags_json, artifact_signature, is_decompiled, created_at, updated_at
+        artifact_id, alias, origin, coordinate, version, binary_jar_path, source_jar_path, repo_url, requested_mapping, mapping_applied, provenance_json, quality_flags_json, artifact_signature, is_decompiled, created_at, updated_at
       ) VALUES (
-        @artifact_id, @origin, @coordinate, @version, @binary_jar_path, @source_jar_path, @repo_url, @requested_mapping, @mapping_applied, @provenance_json, @quality_flags_json, @artifact_signature, @is_decompiled, @created_at, @updated_at
+        @artifact_id, @alias, @origin, @coordinate, @version, @binary_jar_path, @source_jar_path, @repo_url, @requested_mapping, @mapping_applied, @provenance_json, @quality_flags_json, @artifact_signature, @is_decompiled, @created_at, @updated_at
       )
       ON CONFLICT(artifact_id) DO UPDATE SET
+        alias = excluded.alias,
         origin = excluded.origin,
         coordinate = excluded.coordinate,
         version = excluded.version,
@@ -87,9 +90,12 @@ export class ArtifactsRepo {
         is_decompiled = excluded.is_decompiled,
         updated_at = excluded.updated_at
     `);
+        // artifact_id is a 64-char SHA hex; alias is `<type>-<...>-<6charhex>` containing
+        // dashes/letters. The two namespaces cannot collide, so OR-matching is unambiguous.
         this.getStmt = this.db.prepare(`
       SELECT
         artifact_id,
+        alias,
         origin,
         coordinate,
         version,
@@ -105,17 +111,28 @@ export class ArtifactsRepo {
         created_at,
         updated_at
       FROM artifacts
-      WHERE artifact_id = ?
+      WHERE artifact_id = ? OR alias = ?
+      LIMIT 1
     `);
         this.touchStmt = this.db.prepare(`
       UPDATE artifacts
       SET updated_at = ?
       WHERE artifact_id = ?
+    `);
+        // Persists alias on cache-hit / migrated-row paths where upsertArtifact would
+        // otherwise be skipped. Conditional WHERE keeps it idempotent and avoids a
+        // pointless write when alias is already correct.
+        this.setAliasStmt = this.db.prepare(`
+      UPDATE artifacts
+      SET alias = ?
+      WHERE artifact_id = ?
+        AND (alias IS NULL OR alias <> ?)
     `);
         this.deleteStmt = this.db.prepare(`DELETE FROM artifacts WHERE artifact_id = ?`);
         this.listStmt = this.db.prepare(`
       SELECT
         artifact_id,
+        alias,
         origin,
         coordinate,
         version,
@@ -157,6 +174,7 @@ export class ArtifactsRepo {
     upsertArtifact(input) {
         this.upsertStmt.run({
             artifact_id: input.artifactId,
+            alias: input.alias ?? null,
             origin: input.origin,
             coordinate: input.coordinate ?? null,
             version: input.version ?? null,
@@ -173,8 +191,11 @@ export class ArtifactsRepo {
             updated_at: input.timestamp
         });
     }
-    getArtifact(artifactId) {
-        const row = this.getStmt.get([artifactId]);
+    // Accepts either an artifact_id (64-char SHA hex) or a human-readable alias
+    // (e.g. `mc-1.21.10-mojang-merged-5ad2e7`). The two key namespaces cannot
+    // collide because aliases always contain `-` and a non-hex prefix.
+    getArtifact(artifactIdOrAlias) {
+        const row = this.getStmt.get([artifactIdOrAlias, artifactIdOrAlias]);
         if (!row) {
             return undefined;
         }
@@ -183,6 +204,13 @@ export class ArtifactsRepo {
     touchArtifact(artifactId, timestamp) {
         this.touchStmt.run([timestamp, artifactId]);
     }
+    // Backfills or rotates the alias for an existing row. Used by warm-cache
+    // resolveArtifact paths where upsertArtifact is skipped, so a freshly
+    // computed alias still reaches the DB and stays in sync with the value the
+    // caller just received in the response.
+    setAlias(artifactId, alias) {
+        this.setAliasStmt.run([alias, artifactId, alias]);
+    }
     deleteArtifact(artifactId) {
         this.deleteStmt.run([artifactId]);
     }

package/dist/storage/files-repo.d.ts

@@ -43,8 +43,6 @@ export declare class FilesRepo {
     private readonly db;
     private readonly deleteStmt;
     private readonly insertFilesStmt;
-    private readonly insertFtsStmt;
-    private readonly deleteFtsStmt;
     private readonly getContentStmt;
     private readonly listStmt;
     private readonly listRowsStmt;

package/dist/storage/files-repo.js

@@ -125,8 +125,6 @@ export class FilesRepo {
     db;
     deleteStmt;
     insertFilesStmt;
-    insertFtsStmt;
-    deleteFtsStmt;
     getContentStmt;
     listStmt;
     listRowsStmt;
@@ -138,17 +136,10 @@ export class FilesRepo {
         this.db = db;
         this.deleteStmt = this.db.prepare(`
       DELETE FROM files WHERE artifact_id = ?
-    `);
-        this.deleteFtsStmt = this.db.prepare(`
-      DELETE FROM files_fts WHERE artifact_id = ?
     `);
         this.insertFilesStmt = this.db.prepare(`
       INSERT INTO files (artifact_id, file_path, content, content_bytes, content_hash)
       VALUES (?, ?, ?, ?, ?)
-    `);
-        this.insertFtsStmt = this.db.prepare(`
-      INSERT INTO files_fts (artifact_id, file_path, content)
-      VALUES (?, ?, ?)
     `);
         this.getContentStmt = this.db.prepare(`
       SELECT artifact_id, file_path, content, content_bytes, content_hash
@@ -187,7 +178,6 @@ export class FilesRepo {
     }
     clearFilesForArtifact(artifactId) {
         this.deleteStmt.run([artifactId]);
-        this.deleteFtsStmt.run([artifactId]);
     }
     insertFilesForArtifact(artifactId, files) {
         for (const file of files) {
@@ -199,7 +189,6 @@ export class FilesRepo {
                 file.contentBytes,
                 contentHash
             ]);
-            this.insertFtsStmt.run([artifactId, file.filePath, file.content]);
         }
     }
     replaceFilesForArtifact(artifactId, files) {

package/dist/storage/migrations.d.ts

@@ -6,6 +6,6 @@ type MigrationRunner = {
     };
     transaction<T>(fn: () => T): () => T;
 };
-export declare const LATEST_SCHEMA_VERSION = 2;
+export declare const LATEST_SCHEMA_VERSION = 4;
 export declare function runMigrations(db: MigrationRunner): number;
 export {};

package/dist/storage/migrations.js

@@ -1,6 +1,6 @@
 import { createError, ERROR_CODES } from "../errors.js";
-import { SCHEMA_V1_STATEMENTS, SCHEMA_V2_STATEMENTS } from "./schema.js";
-export const LATEST_SCHEMA_VERSION = 2;
+import { SCHEMA_V1_STATEMENTS, SCHEMA_V2_STATEMENTS, SCHEMA_V3_STATEMENTS, SCHEMA_V4_STATEMENTS } from "./schema.js";
+export const LATEST_SCHEMA_VERSION = 4;
 const migrations = [
     {
         version: 1,
@@ -9,6 +9,14 @@ const migrations = [
     {
         version: 2,
         statements: SCHEMA_V2_STATEMENTS
+    },
+    {
+        version: 3,
+        statements: SCHEMA_V3_STATEMENTS
+    },
+    {
+        version: 4,
+        statements: SCHEMA_V4_STATEMENTS
     }
 ];
 function selectSchemaVersion(tx) {

package/dist/storage/schema.d.ts

@@ -1,2 +1,4 @@
 export declare const SCHEMA_V1_STATEMENTS: string[];
 export declare const SCHEMA_V2_STATEMENTS: string[];
+export declare const SCHEMA_V4_STATEMENTS: string[];
+export declare const SCHEMA_V3_STATEMENTS: string[];

package/dist/storage/schema.js

@@ -164,4 +164,29 @@ export const SCHEMA_V2_STATEMENTS = [
     `DELETE FROM artifact_index_meta`,
     `DELETE FROM artifacts`
 ];
+export const SCHEMA_V4_STATEMENTS = [
+    `ALTER TABLE artifacts ADD COLUMN alias TEXT`,
+    `CREATE UNIQUE INDEX IF NOT EXISTS idx_artifacts_alias ON artifacts(alias) WHERE alias IS NOT NULL`
+];
+export const SCHEMA_V3_STATEMENTS = [
+    `DROP TABLE IF EXISTS files_fts`,
+    `CREATE VIRTUAL TABLE files_fts USING fts5(
+    artifact_id UNINDEXED,
+    file_path,
+    content,
+    content='files',
+    content_rowid='rowid',
+    tokenize = 'unicode61 separators ''._$'''
+  )`,
+    `CREATE TRIGGER IF NOT EXISTS trg_fts_insert AFTER INSERT ON files BEGIN
+    INSERT INTO files_fts(rowid, artifact_id, file_path, content)
+    VALUES (NEW.rowid, NEW.artifact_id, NEW.file_path, NEW.content);
+  END`,
+    // No UPDATE trigger: FilesRepo uses delete+insert, never updates rows in place.
+    `CREATE TRIGGER IF NOT EXISTS trg_fts_delete BEFORE DELETE ON files BEGIN
+    INSERT INTO files_fts(files_fts, rowid, artifact_id, file_path, content)
+    VALUES ('delete', OLD.rowid, OLD.artifact_id, OLD.file_path, OLD.content);
+  END`,
+    `INSERT INTO files_fts(files_fts) VALUES('rebuild')`
+];
 //# sourceMappingURL=schema.js.map

package/dist/types.d.ts

@@ -5,12 +5,14 @@ export type RuntimeValidationNamespace = SourceMapping | AccessTransformerNamesp
 export type MappingSourcePriority = "loom-first" | "maven-first";
 export type ArtifactTargetKind = "version" | "jar" | "coordinate";
 export type ArtifactScope = "vanilla" | "merged" | "loader";
+export type MappingVariant = "pass" | "mojang-remapped";
 export interface SourceTargetInput {
     kind: ArtifactTargetKind;
     value: string;
 }
 export interface ResolvedSourceArtifact {
     artifactId: string;
+    artifactAlias?: string;
     artifactSignature: string;
     origin: SourceOrigin;
     binaryJarPath?: string;
@@ -102,6 +104,7 @@ export interface SourceSearchHit {
 }
 export interface ArtifactRow {
     artifactId: string;
+    alias: string | undefined;
     origin: SourceOrigin;
     coordinate: string | undefined;
     version: string | undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adhisang/minecraft-modding-mcp",
-  "version": "3.2.0",
+  "version": "4.0.0",
   "description": "MCP server with utilities for Minecraft modding workflows",
   "type": "module",
   "main": "dist/index.js",
@@ -23,6 +23,8 @@
     "start": "node dist/cli.js",
     "check": "tsc --noEmit -p tsconfig.json",
     "test": "node --test --import tsx tests/*.test.ts",
+    "test:file": "node --test --import tsx",
+    "test:grep": "node --test --import tsx --test-name-pattern",
     "test:coverage": "node --test --import tsx --experimental-test-coverage --test-coverage-lines=80 --test-coverage-branches=70 --test-coverage-functions=80 tests/*.test.ts",
     "test:coverage:lcov": "node --input-type=module -e \"import { mkdirSync } from 'node:fs'; mkdirSync('coverage', { recursive: true });\" && node --test --import tsx --experimental-test-coverage --test-reporter=lcov --test-reporter-destination=coverage/lcov.info --test-coverage-lines=80 --test-coverage-branches=70 --test-coverage-functions=80 tests/*.test.ts",
     "test:perf": "node --test --test-concurrency=1 --import tsx tests/perf/*.perf.ts",