@gesslar/lpc-mcp 0.1.4 → 1.0.3

package/.github/workflows/Quality.yaml

@@ -0,0 +1,192 @@
+name: CodeQL, Linting, Testing
+
+on:
+  workflow_dispatch:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+  schedule:
+    - cron: "20 14 * * 1"
+
+jobs:
+  ConfigureWorkflow:
+    runs-on: ubuntu-latest
+    permissions: {}
+    outputs:
+      perform_testing: ${{ steps.cfg.outputs.perform_testing }}
+      perform_linting: ${{ steps.cfg.outputs.perform_linting }}
+      package_manager: ${{ steps.cfg.outputs.package_manager }}
+      pm_install_cmd: ${{ steps.cfg.outputs.pm_install_cmd }}
+
+    steps:
+      - name: Configure workflow
+        id: cfg
+        run: |+
+          ### PACKAGE MANAGER #################################################
+          #
+          # Set the package manager to use: 'npm' or 'pnpm'
+          # This will configure the appropriate setup and commands throughout
+          # the workflow.
+
+          PACKAGE_MANAGER="pnpm"
+          echo "package_manager=${PACKAGE_MANAGER}" >> "$GITHUB_OUTPUT"
+
+          # Set package manager specific commands and versions
+          if [ "$PACKAGE_MANAGER" = "pnpm" ]; then
+            echo "pm_install_cmd=pnpm install --frozen-lockfile" >> "$GITHUB_OUTPUT"
+          else
+            echo "pm_install_cmd=npm ci" >> "$GITHUB_OUTPUT"
+          fi
+
+          ### LINTING #########################################################
+          #
+          # If we need to perform linting, set the value to 'yes'. Any other
+          # value will skip linting.
+
+          echo "perform_linting=yes" >> "$GITHUB_OUTPUT"
+
+          ### TESTING #########################################################
+          #
+          # If we need to perform testing, set the value to 'yes'. Any other
+          # value will skip testing.
+
+          echo "perform_testing=no" >> "$GITHUB_OUTPUT"
+
+  Lint:
+    needs: ConfigureWorkflow
+    runs-on: ubuntu-latest
+
+    permissions:
+      contents: read
+
+    strategy:
+      matrix:
+        node-version: [22.x]
+
+    steps:
+      - name: Check out repository
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_linting == 'yes' }}
+        uses: actions/checkout@v6
+
+      - name: Install pnpm
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_linting == 'yes' && needs.ConfigureWorkflow.outputs.package_manager == 'pnpm' }}
+        uses: pnpm/action-setup@v4
+
+      - name: Using Node v${{ matrix.node-version }}
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_linting == 'yes' }}
+        uses: actions/setup-node@v6
+        with:
+          node-version: ${{ matrix.node-version }}
+          cache: ${{ needs.ConfigureWorkflow.outputs.package_manager }}
+
+      - name: Install dependencies
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_linting == 'yes' }}
+        run: ${{ needs.ConfigureWorkflow.outputs.pm_install_cmd }}
+
+      - name: Perform lint check
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_linting == 'yes' }}
+        run: ${{ needs.ConfigureWorkflow.outputs.package_manager }} run lint
+
+  Test:
+    needs: ConfigureWorkflow
+    runs-on: ubuntu-latest
+
+    permissions:
+      contents: read
+
+    strategy:
+      matrix:
+        node-version: [22.x]
+
+    steps:
+      - name: Check out repository
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_testing == 'yes' }}
+        uses: actions/checkout@v6
+
+      - name: Install pnpm
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_testing == 'yes' && needs.ConfigureWorkflow.outputs.package_manager == 'pnpm' }}
+        uses: pnpm/action-setup@v4
+
+      - name: Using Node v${{ matrix.node-version }}
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_testing == 'yes' }}
+        uses: actions/setup-node@v6
+        with:
+          node-version: ${{ matrix.node-version }}
+          cache: ${{ needs.ConfigureWorkflow.outputs.package_manager }}
+
+      - name: Install dependencies
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_testing == 'yes' }}
+        run: ${{ needs.ConfigureWorkflow.outputs.pm_install_cmd }}
+
+      - name: Definitely don't cheat on tests
+        if: ${{ needs.ConfigureWorkflow.outputs.perform_testing == 'yes' }}
+        run: ${{ needs.ConfigureWorkflow.outputs.package_manager }} test
+
+  Analyze:
+    name: Analyze (${{ matrix.language }})
+    needs: ["Lint", "Test"]
+    runs-on: ubuntu-latest
+    permissions:
+      # required for all workflows
+      security-events: write
+
+      # required to fetch internal or private CodeQL packs
+      packages: read
+
+      # only required for workflows in private repositories
+      actions: read
+      contents: read
+
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - language: actions
+            build-mode: none
+          - language: javascript-typescript
+            build-mode: none
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      # Add any setup steps before running the `github/codeql-action/init` action.
+      # This includes steps like installing compilers or runtimes (`actions/setup-node`
+      # or others). This is typically only required for manual builds.
+      # - name: Setup runtime (example)
+      #   uses: actions/setup-example@v1
+
+      # Initializes the CodeQL tools for scanning.
+      - name: Initialize CodeQL
+        uses: github/codeql-action/init@v4
+        with:
+          languages: ${{ matrix.language }}
+          build-mode: ${{ matrix.build-mode }}
+          # If you wish to specify custom queries, you can do so here or in a config file.
+          # By default, queries listed here will override any specified in a config file.
+          # Prefix the list here with "+" to use these queries and those in the config file.
+
+          # For more details on CodeQL's query packs, refer to: https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-[...]
+          # queries: security-extended,security-and-quality
+
+      # If the analyze step fails for one of the languages you are analyzing with
+      # "We were unable to automatically build your code", modify the matrix above
+      # to set the build mode to "manual" for that language. Then modify this step
+      # to build your code.
+      # ℹ️ Command-line programs to run using the OS shell.
+      # 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun
+      - name: Run manual build steps
+        if: matrix.build-mode == 'manual'
+        shell: bash
+        run: |
+          echo 'If you are using a "manual" build mode for one or more of the' \
+            'languages you are analyzing, replace this with the commands to build' \
+            'your code, for example:'
+          echo '  make bootstrap'
+          echo '  make release'
+          exit 1
+
+      - name: Perform CodeQL Analysis
+        uses: github/codeql-action/analyze@v4
+        with:
+          category: "/language:${{matrix.language}}"

package/.github/workflows/Release.yaml

@@ -0,0 +1,214 @@
+name: Release
+
+permissions:
+  contents: read
+
+on:
+  pull_request:
+    types:
+      - closed
+    branches:
+      - main
+
+jobs:
+  ConfigureWorkflow:
+    runs-on: ubuntu-latest
+
+    outputs:
+      quality_check: ${{ steps.cfg.outputs.quality_check }}
+
+    steps:
+      - name: Configure workflow
+        id: cfg
+        run: |+
+          # If we need to wait for a workflow to have completed before
+          # we get here, name the workflow here so that it is checked
+          # against later. Or just leave empty, implicitly telling this
+          # workflow not to wait, just keep swimming.
+
+          echo "quality_check='CodeQL, Linting, Testing'" >> "$GITHUB_OUTPUT"
+
+  WaitForQuality:
+    needs: ConfigureWorkflow
+    runs-on: ubuntu-latest
+    if: ${{ github.event.pull_request.merged == true }}
+
+    steps:
+      - name: Wait for Quality workflow to complete
+        if: ${{ needs.ConfigureWorkflow.outputs.quality_check != '' }}
+        uses: lewagon/wait-on-check-action@v1.3.4
+        with:
+          ref: ${{ github.ref }}
+          running-workflow-name: ${{ needs.ConfigureWorkflow.outputs.quality_check }}
+          repo-token: ${{ secrets.GITHUB_TOKEN }}
+          wait-interval: 10
+
+      - name: Skipping wait (no quality_check configured)
+        if: ${{ needs.ConfigureWorkflow.outputs.quality_check == '' }}
+        run: echo "No quality check configured; continuing."
+
+  DetermineVersion:
+    needs: WaitForQuality
+    runs-on: ubuntu-latest
+    if: ${{ github.event.pull_request.merged == true }}
+
+    outputs:
+      version: ${{ steps.version.outputs.VERSION }}
+
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Determine version
+        id: check_version
+        uses: gesslar/new-version-questionmark@v1.5.5
+        with:
+          source: package.json
+          version_pattern: "^v\\d+\\.\\d+\\.\\d+$"
+
+      - name: Show result
+        id: version
+        run: |+
+          UPDATED_VERSION="${{ steps.check_version.outputs.updated_version }}"
+
+          if [[ "$UPDATED_VERSION" == "no changes" ]]; then
+            echo "VERSION=🙅🏻‍♂️" >>  "$GITHUB_OUTPUT"
+          else
+            echo "VERSION=${UPDATED_VERSION}" >> "$GITHUB_OUTPUT"
+          fi
+
+  CreateTag:
+    needs: DetermineVersion
+    if: ${{ needs.DetermineVersion.outputs.version != '🙅🏻‍♂️' }}
+    runs-on: ubuntu-latest
+
+    permissions:
+      contents: write
+
+    env:
+      GIT_USER_NAME: "github-actions"
+      GIT_USER_EMAIL: "github-actions@github.com"
+
+    outputs:
+      new_tag: ${{ steps.check_tag.outputs.new_tag }}
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v6
+
+      - name: Setup Git Variables
+        run: |+
+          echo "GIT_USER_NAME=${GIT_USER_NAME}" >> $GITHUB_ENV
+          echo "GIT_USER_EMAIL=${GIT_USER_EMAIL}" >> $GITHUB_ENV
+          git config user.name "${GIT_USER_NAME}"
+          git config user.email "${GIT_USER_EMAIL}"
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v6
+        with:
+          node-version: "22"
+
+      - name: Install dependencies
+        run: npm install
+
+      - name: Read version from package.json
+        id: get_version
+        run: |
+          CURRENT_VERSION=${{ needs.DetermineVersion.outputs.version }}
+
+      - name: Fetch all tags
+        run: git fetch --tags
+
+      - name: Check for existing tag and create if it does not exist
+        id: check_tag
+        run: |+
+          CURRENT_VERSION="${{ needs.DetermineVersion.outputs.version }}"
+          TAG_EXISTS=$(git tag -l "v${CURRENT_VERSION}")
+
+          echo "Value of TAG_EXISTS: $TAG_EXISTS"
+
+          if [ -z "$TAG_EXISTS" ]; then
+            echo "Tag does not exist for version v${CURRENT_VERSION}. Creating tag."
+            git tag "v${CURRENT_VERSION}"
+            git push origin "v${CURRENT_VERSION}"
+            echo "new_tag=v${CURRENT_VERSION}" >> $GITHUB_OUTPUT
+          else
+            echo "Tag already exists for version v${CURRENT_VERSION}. Skipping tag creation."
+            echo "new_tag=🙅🏻‍♂️" >> $GITHUB_OUTPUT
+            exit 0
+          fi
+
+  PrepareGitHubRelease:
+    needs: CreateTag
+    runs-on: ubuntu-latest
+    if: ${{ needs.CreateTag.outputs.new_tag != '🙅🏻‍♂️' }}
+
+    permissions:
+      contents: write
+
+    env:
+      NEW_TAG: ${{ needs.CreateTag.outputs.new_tag }}
+      GIT_USER_NAME: "github-actions"
+      GIT_USER_EMAIL: "github-actions@github.com"
+
+    steps:
+      - name: Setup New Tag
+        run: |
+          echo "NEW_TAG=${NEW_TAG}" >> $GITHUB_ENV
+
+      - name: Checkout code
+        uses: actions/checkout@v6
+
+      - name: Setup Git Variables
+        run: |
+          echo "GIT_USER_NAME=${GIT_USER_NAME}" >> $GITHUB_ENV
+          echo "GIT_USER_EMAIL=${GIT_USER_EMAIL}" >> $GITHUB_ENV
+          git config user.name "${GIT_USER_NAME}"
+          git config user.email "${GIT_USER_EMAIL}"
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v6
+        with:
+          node-version: "22"
+
+      - name: Install dependencies
+        run: npm install
+
+      - name: Build artefact
+        run: |
+          ARTEFACT=$(npm pack 2>/dev/null | tail -1)
+          echo "ARTEFACT=${ARTEFACT}" >> $GITHUB_ENV
+
+      - name: Create Release
+        uses: ncipollo/release-action@v1
+        with:
+          tag: ${{ env.NEW_TAG }}
+          name: Release ${{ env.NEW_TAG }}
+          draft: false
+          prerelease: false
+          artifacts: ${{ env.ARTEFACT }}
+          generateReleaseNotes: true
+
+  PublishToNPM:
+    needs: DetermineVersion
+    if: ${{ github.event.pull_request.merged == true && needs.DetermineVersion.outputs.version != '🙅🏻‍♂️' }}
+    runs-on: ubuntu-latest
+
+    permissions:
+      contents: write
+    env:
+      TOKEN: ${{ secrets.NPM_GITHUB_CD_ACCESS_TOKEN }}
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v6
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v6
+        with:
+          node-version: "22"
+
+      - name: Ship it
+        run: |
+          npm publish --access public --//registry.npmjs.org/:_authToken=${TOKEN}

package/README.md

@@ -30,10 +30,10 @@ All through conversation, powered by actual code intelligence instead of pattern
 
 ### 1. Install Node.js
 
-Node.js 16+ required:
+Node.js 20+ required:
 
 ```bash
-node --version  # Should be v16.0.0 or higher
+node --version  # Should be v20.0.0 or higher
 ```
 
 ### 2. Install the LPC Language Server Extension

package/eslint.config.js

@@ -1,167 +1,11 @@
-import js from "@eslint/js"
-import jsdoc from "eslint-plugin-jsdoc"
-import stylistic from "@stylistic/eslint-plugin"
-import globals from "globals"
+import uglify from "@gesslar/uglier"
 
 export default [
-  js.configs.recommended,
-  jsdoc.configs['flat/recommended'], {
-    name: "gesslar/uglier/ignores",
-    ignores: [],
-  }, {
-    name: "gesslar/uglier/languageOptions",
-    languageOptions: {
-      ecmaVersion: "latest",
-      sourceType: "module",
-      globals: {
-        ...globals.node,
-        fetch: "readonly",
-        Headers: "readonly",
-      },
-    },
-  },
-  // Add override for webview files to include browser globals
-  {
-    name: "gesslar/uglier/webview-env",
-    files: ["src/webview/**/*.{js,mjs,cjs}"],
-    languageOptions: {
-      globals: {
-        ...globals.browser,
-        acquireVsCodeApi: "readonly"
-      }
-    }
-  },
-  // Add override for .cjs files to treat as CommonJS
-  {
-    name: "gesslar/uglier/cjs-override",
-    files: ["src/**/*.cjs"],
-    languageOptions: {
-      sourceType: "script",
-      ecmaVersion: 2021
-    },
-  },
-  // Add override for .mjs files to treat as ES modules
-  {
-    name: "gesslar/uglier/mjs-override",
-    files: ["src/**/*.mjs"],
-    languageOptions: {
-      sourceType: "module",
-      ecmaVersion: 2021
-    }
-  },
-  {
-    name: "gesslar/uglier/lints-js",
-    files: ["{work,src}/**/*.{mjs,cjs,js}"],
-    plugins: {
-      "@stylistic": stylistic,
-    },
-    rules: {
-      "@stylistic/arrow-parens": ["error", "as-needed"],
-      "@stylistic/arrow-spacing": ["error", { before: true, after: true }],
-      "@stylistic/brace-style": ["error", "1tbs", {allowSingleLine: false}],
-      "@stylistic/nonblock-statement-body-position": ["error", "below"],
-      "@stylistic/padding-line-between-statements": [
-        "error",
-        {blankLine: "always",   prev: "if", next: "*"},
-        {blankLine: "always",   prev: "*", next: "return"},
-        {blankLine: "always",   prev: "while", next: "*"},
-        {blankLine: "always",   prev: "for", next: "*"},
-        {blankLine: "always",   prev: "switch", next: "*"},
-        {blankLine: "always",   prev: "do", next: "*"},
-        // {blankLine: "always",   prev: ["const", "let", "var"], next: "*"},
-        // {blankLine: "any",      prev: ["const", "let", "var"], next: ["const", "let", "var"]},
-        {blankLine: "always",   prev: "directive", next: "*" },
-        {blankLine: "any",      prev: "directive", next: "directive" },
-      ],
-      "@stylistic/eol-last": ["error", "always"],
-      "@stylistic/indent": ["error", 2, {
-        SwitchCase: 1 // Indents `case` statements one level deeper than `switch`
-      }],
-      "@stylistic/key-spacing": ["error", { beforeColon: false, afterColon: true }],
-      "@stylistic/keyword-spacing": ["error", {
-        before: false,
-        after: true,
-        overrides: {
-          // Control statements
-          return:  { before: true, after: true },
-          if:      { after: false },
-          else:    { before: true, after: true },
-          for:     { after: false },
-          while:   { before: true, after: false },
-          do:      { after: true },
-          switch:  { after: false },
-          case:    { before: true, after: true },
-          throw:   { before: true, after: false } ,
-
-          // Keywords
-          as:      { before: true, after: true },
-          of:      { before: true, after: true },
-          from:    { before: true, after: true },
-          async:   { before: true, after: true },
-          await:   { before: true, after: false },
-          class:   { before: true, after: true },
-          const:   { before: true, after: true },
-          let:     { before: true, after: true },
-          var:     { before: true, after: true },
-
-          // Exception handling
-          catch:   { before: true, after: true },
-          finally: { before: true, after: true },
-        }
-      }],
-      // Blocks
-      "@stylistic/space-before-blocks": ["error", "always"],
-      "@stylistic/max-len": ["warn", {
-        code: 80,
-        ignoreComments: true,
-        ignoreUrls: true,
-        ignoreStrings: true,
-        ignoreTemplateLiterals: true,
-        ignoreRegExpLiterals: true,
-        tabWidth: 2
-      }],
-      "@stylistic/no-tabs": "error",
-      "@stylistic/no-trailing-spaces": ["error"],
-      "@stylistic/object-curly-spacing": ["error", "never", {
-        objectsInObjects: false,
-        arraysInObjects: false
-      }],
-      "@stylistic/quotes": ["error", "double", {
-        avoidEscape: true,
-        allowTemplateLiterals: "always"
-      }],
-      "@stylistic/semi": ["error", "never"],
-      "@stylistic/space-before-function-paren": ["error", "never"],
-      "@stylistic/yield-star-spacing": ["error", { before: true, after: false }],
-      "constructor-super": "error",
-      "no-unexpected-multiline": "error",
-      "no-unused-vars": ["error", {
-        caughtErrors: "all",
-        caughtErrorsIgnorePattern: "^_+",
-        argsIgnorePattern: "^_+",
-        destructuredArrayIgnorePattern: "^_+",
-        varsIgnorePattern: "^_+"
-      }],
-      "no-useless-assignment": "error",
-      "prefer-const": "error",
-      "@stylistic/no-multiple-empty-lines": ["error", { max: 1 }],
-      "@stylistic/array-bracket-spacing": ["error", "never"],
-    }
-  },
-  {
-    name: "gesslar/uglier/lints-jsdoc",
-    files: ["{work,src}/**/*.{mjs,cjs,js}"],
-    plugins: {
-      jsdoc,
-    },
-    rules: {
-      "jsdoc/require-description": "error",
-      "jsdoc/tag-lines": ["error", "any", {"startLines":1}],
-      "jsdoc/require-jsdoc": ["error", { publicOnly: true }],
-      "jsdoc/check-tag-names": "error",
-      "jsdoc/check-types": "error",
-      "jsdoc/require-param-type": "error",
-      "jsdoc/require-returns-type": "error"
-    }
-  }
+  ...uglify({
+    with: [
+      "lints-js", // default files: ["**/*.{js,mjs,cjs}"]
+      "lints-jsdoc", // default files: ["**/*.{js,mjs,cjs}"]
+      "node", // default files: ["**/*.{js,mjs,cjs}"]
+    ]
+  })
 ]

package/package.json

@@ -1,43 +1,52 @@
 {
   "name": "@gesslar/lpc-mcp",
-  "version": "0.1.4",
   "description": "MCP server for LPC language server",
+  "author": "gesslar",
+  "version": "1.0.3",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/gesslar/lpc-mcp.git"
+  },
   "main": "index.js",
   "type": "module",
   "bin": {
     "lpc-mcp": "./src/index.js"
   },
   "engines": {
-    "node": ">=20"
+    "node": ">=22"
   },
+  "packageManager": "pnpm@10.26.2",
   "scripts": {
     "start": "node src/index.js",
     "types:build": "tsc -p tsconfig.types.json",
     "lint": "eslint src/",
     "lint:fix": "eslint src/ --fix",
-    "submit": "npm publish --access public",
-    "update": "npx npm-check-updates -u && npm install",
-    "pr": "gt submit --publish --restack --ai -m",
-    "patch": "npm version patch",
-    "minor": "npm version minor",
-    "major": "npm version major"
+    "submit": "pnpm publish --access public --//registry.npmjs.org/:_authToken=\"${NPM_ACCESS_TOKEN}\"",
+    "update": "pnpm up -L",
+    "pr": "gt submit -p --ai",
+    "patch": "pnpm version patch",
+    "minor": "pnpm version minor",
+    "major": "pnpm version major"
   },
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/gesslar/lpc-mcp.git"
-  },
-  "keywords": [],
-  "author": "gesslar",
+  "keywords": [
+    "fluffos",
+    "lpc",
+    "mud",
+    "mcp",
+    "model-context-protocol",
+    "shlemiel",
+    "schlemazel",
+    "hasenpfeffer",
+    "incorporated"
+  ],
   "license": "Unlicense",
   "homepage": "https://github.com/gesslar/lpc-mcp#readme",
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.20.2",
+    "@modelcontextprotocol/sdk": "^1.25.1",
     "vscode-jsonrpc": "^8.2.1"
   },
   "devDependencies": {
-    "@eslint/js": "^9.39.0",
-    "@stylistic/eslint-plugin": "^5.5.0",
-    "eslint-plugin-jsdoc": "^61.1.11",
-    "globals": "^16.5.0"
+    "@gesslar/uglier": "^0.0.9",
+    "eslint": "^9.39.2"
   }
 }

package/test.txt

@@ -0,0 +1,3 @@
+test\n
+test\n
+test\n

package/.github/workflows/ci.yml

@@ -1,33 +0,0 @@
-name: Giddyup
-permissions:
-  contents: read
-
-on:
-  push:
-    branches: [main]
-  pull_request:
-    branches: [main]
-
-jobs:
-  cowyboysounds:
-    runs-on: ubuntu-latest
-
-    strategy:
-      matrix:
-        node-version: [20.x, 22.x]
-
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-
-      - name: Setup Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v4
-        with:
-          node-version: ${{ matrix.node-version }}
-          cache: "npm"
-
-      - name: Install dependencies
-        run: npm ci
-
-      - name: Run ESLint
-        run: npm run lint