@reinteractive/rails-insight 1.0.2 → 1.0.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reinteractive/rails-insight",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "Rails-aware codebase indexer — MCP server for AI agents",
   "type": "module",
   "bin": {

package/src/core/patterns/route.js

@@ -18,8 +18,9 @@ export const ROUTE_PATTERNS = {
   member: /^\s*member\s+do/m,
   collection: /^\s*collection\s+do/m,
   draw: /^\s*draw\s*\(?:?(\w+)\)?/m,
-  only: /only:\s*\[([^\]]+)\]/,
-  except: /except:\s*\[([^\]]+)\]/,
+  only: /only:\s*(?:\[([^\]]+)\]|:([\w]+))/,
+  except: /except:\s*(?:\[([^\]]+)\]|:([\w]+))/,
+  httpVerbSymbol: /^\s*(?:get|post|put|patch|delete)\s+:(\w+)/m,
   defaults: /defaults:\s*\{([^}]+)\}/,
   healthCheck: /^\s*get\s+['"]up['"]/m,
   direct: /^\s*direct\s*\(:(\w+)\)/m,

package/src/core/patterns/schema.js

@@ -10,7 +10,7 @@ export const SCHEMA_PATTERNS = {
     /^\s*t\.(?:references|belongs_to)\s+['"]?:?(\w+)['"]?(?:,\s*(.+))?/m,
   timestamps: /^\s*t\.timestamps/m,
   index:
-    /^\s*(?:t\.index|add_index)\s+(?:\[([^\]]+)\]|['"](\w+)['"]),?\s*(.+)?/m,
+    /^\s*(?:t\.index|add_index)\s+(?:\[([^\]]+)\]|['"]([^'"]+)['"]),?\s*(.+)?/m,
   foreignKey:
     /^\s*add_foreign_key\s+['"](\w+)['"],\s*['"](\w+)['"](?:,\s*(.+))?/m,
   checkConstraint:

package/src/extractors/routes.js

@@ -112,7 +112,8 @@ function parseRouteContent(content, result, provider, namespaceStack) {
       let actions = ['show', 'new', 'create', 'edit', 'update', 'destroy']
       const onlyMatch = options.match(ROUTE_PATTERNS.only)
       if (onlyMatch) {
-        actions = onlyMatch[1].match(/:(\w+)/g)?.map((a) => a.slice(1)) || []
+        const raw = onlyMatch[1] ?? `:${onlyMatch[2]}`
+        actions = raw.match(/:(\w+)/g)?.map((a) => a.slice(1)) || []
       }
 
       const entry = {
@@ -151,12 +152,13 @@ function parseRouteContent(content, result, provider, namespaceStack) {
       ]
       const onlyMatch = options.match(ROUTE_PATTERNS.only)
       if (onlyMatch) {
-        actions = onlyMatch[1].match(/:(\w+)/g)?.map((a) => a.slice(1)) || []
+        const raw = onlyMatch[1] ?? `:${onlyMatch[2]}`
+        actions = raw.match(/:(\w+)/g)?.map((a) => a.slice(1)) || []
       }
       const exceptMatch = options.match(ROUTE_PATTERNS.except)
       if (exceptMatch) {
-        const except =
-          exceptMatch[1].match(/:(\w+)/g)?.map((a) => a.slice(1)) || []
+        const raw = exceptMatch[1] ?? `:${exceptMatch[2]}`
+        const except = raw.match(/:(\w+)/g)?.map((a) => a.slice(1)) || []
         actions = actions.filter((a) => !except.includes(a))
       }
 
@@ -208,6 +210,20 @@ function parseRouteContent(content, result, provider, namespaceStack) {
       continue
     }
 
+    // Symbol-form verb routes inside member/collection blocks: `get :action_name`
+    if (inMember || inCollection) {
+      const symbolVerbMatch = trimmed.match(ROUTE_PATTERNS.httpVerbSymbol)
+      if (symbolVerbMatch) {
+        const action = symbolVerbMatch[1]
+        const currentResource = resourceStack[resourceStack.length - 1]
+        if (currentResource) {
+          if (inMember) currentResource.member_routes.push(action)
+          else currentResource.collection_routes.push(action)
+        }
+        continue
+      }
+    }
+
     // HTTP verb routes
     const verbMatch = trimmed.match(ROUTE_PATTERNS.httpVerb)
     if (verbMatch) {

package/src/extractors/schema.js

@@ -149,16 +149,22 @@ export function extractSchema(provider) {
     // Index
     const indexMatch = trimmed.match(SCHEMA_PATTERNS.index)
     if (indexMatch) {
+      const isExpression = indexMatch[1] === undefined && indexMatch[2] !== undefined && /[^a-zA-Z0-9_]/.test(indexMatch[2])
       const columns = indexMatch[1]
         ? indexMatch[1]
             .match(/['"](\w+)['"]/g)
             ?.map((c) => c.replace(/['"]/g, '')) || []
-        : [indexMatch[2]]
+        : isExpression
+          ? []
+          : [indexMatch[2]]
       const opts = indexMatch[3] || ''
       currentTable.indexes.push({
         columns,
+        ...(isExpression ? { expression: indexMatch[2] } : {}),
         unique: /unique:\s*true/.test(opts),
         name: opts.match(/name:\s*['"]([^'"]+)['"]/)?.[1] || null,
+        where: opts.match(/where:\s*['"]([^'"]+)['"]/)?.[1] || null,
+        using: opts.match(/using:\s*:(\w+)/)?.[1] || null,
       })
       continue
     }

package/src/tools/handlers/get-schema.js

@@ -15,9 +15,10 @@ export function register(server, state) {
       const schema = state.index.extractions?.schema || {}
       const models = state.index.extractions?.models || {}
 
-      // Add model ↔ table mapping
+      // Add model ↔ table mapping (concrete AR models only, not concerns/modules)
       const modelTableMap = {}
       for (const [modelName, modelData] of Object.entries(models)) {
+        if (modelData.type === 'concern') continue
         const tableName = modelData.table_name || toTableName(modelName)
         modelTableMap[modelName] = tableName
       }