ts-procedures 5.9.1 → 5.10.2

package/src/stack-utils.ts

@@ -0,0 +1,129 @@
+/**
+ * Represents a specific location in source code where a procedure was defined.
+ */
+export type DefinitionLocation = {
+  file: string
+  line: number
+  column: number
+  raw: string
+}
+
+/**
+ * Contains information about where a procedure was defined.
+ */
+export type DefinitionInfo = {
+  definedAt?: DefinitionLocation
+  definitionStack?: string
+}
+
+/**
+ * Internal ts-procedures files that should be skipped when finding user code.
+ * Only skip the core library files, not test files or user code.
+ */
+const INTERNAL_FILES = [
+  '/index.ts',
+  '/index.js',
+  '/errors.ts',
+  '/errors.js',
+  '/stack-utils.ts',
+  '/stack-utils.js',
+  '/compute-schema.ts',
+  '/compute-schema.js',
+  '/parser.ts',
+  '/parser.js',
+]
+
+/**
+ * Captures the stack trace at the call site and extracts the definition location.
+ * Finds the first stack frame outside of ts-procedures internal files.
+ */
+export function captureDefinitionInfo(): DefinitionInfo {
+  const err = new Error()
+  const stack = err.stack
+
+  if (!stack) {
+    return {}
+  }
+
+  const lines = stack.split('\n')
+
+  // Find the first frame that's not from ts-procedures internals
+  // Skip the first line (Error message) and frames from this module
+  let userFrame: string | undefined
+
+  for (let i = 1; i < lines.length; i++) {
+    const rawLine = lines[i]
+    if (!rawLine) continue
+    const line = rawLine.trim()
+
+    // Skip empty or invalid frames
+    if (!line.startsWith('at ')) {
+      continue
+    }
+
+    // Skip frames from ts-procedures internal source files
+    const isInternalFile = INTERNAL_FILES.some(file => line.includes(file))
+    if (isInternalFile) {
+      continue
+    }
+
+    // Skip frames from ts-procedures in node_modules (when used as a dependency)
+    if (line.includes('/node_modules/ts-procedures/') || line.includes('\\node_modules\\ts-procedures\\')) {
+      continue
+    }
+
+    // Skip internal node frames
+    if (line.includes('node:') || line.startsWith('at Module.') || line.startsWith('at Object.<anonymous> (node:')) {
+      continue
+    }
+
+    userFrame = line
+    break
+  }
+
+  if (!userFrame) {
+    return { definitionStack: stack }
+  }
+
+  const definedAt = parseStackFrame(userFrame)
+
+  return {
+    definedAt,
+    definitionStack: stack,
+  }
+}
+
+/**
+ * Parses a V8 stack frame line to extract file, line, and column info.
+ * Handles formats like:
+ * - "at Object.<anonymous> (/path/to/file.ts:10:5)"
+ * - "at functionName (/path/to/file.ts:10:5)"
+ * - "at /path/to/file.ts:10:5"
+ */
+function parseStackFrame(frame: string): DefinitionLocation | undefined {
+  // Match patterns like "(path:line:column)" or just "path:line:column"
+  const match = frame.match(/\(([^)]+):(\d+):(\d+)\)$/) || frame.match(/at ([^:]+):(\d+):(\d+)$/)
+
+  if (match && match[1] && match[2] && match[3]) {
+    return {
+      file: match[1],
+      line: parseInt(match[2], 10),
+      column: parseInt(match[3], 10),
+      raw: frame,
+    }
+  }
+
+  return undefined
+}
+
+/**
+ * Formats definition info for appending to error stacks.
+ */
+export function formatDefinitionInfo(info: DefinitionInfo, procedureName: string): string | undefined {
+  if (!info.definedAt) {
+    return undefined
+  }
+
+  const { file, line, column } = info.definedAt
+  return `\n--- Procedure "${procedureName}" defined at ---\n    at ${file}:${line}:${column}`
+}

package/agent_config/claude-code/skills/review/SKILL.md

@@ -1,53 +0,0 @@
----
-name: review
-description: "Review code for ts-procedures pattern adherence. Usage: /ts-procedures:review <path>"
-invocable_by:
-  - user
-  - model
-user_instructions: |
-  Usage: /ts-procedures:review <path>
-
-  Reviews files at the given path for ts-procedures pattern adherence.
-  Accepts a file path or directory.
-
-  Examples:
-    /ts-procedures:review src/procedures/
-    /ts-procedures:review src/procedures/GetUser.procedure.ts
----
-
-# Review ts-procedures Code
-
-Parse `$ARGUMENTS` as a file or directory path. If a directory, review all `.ts` and `.tsx` files within it.
-
-## Instructions
-
-1. Read the target file(s).
-2. Identify ts-procedures imports (`ts-procedures`, `ts-procedures/express-rpc`, `ts-procedures/hono-rpc`, `ts-procedures/hono-stream`, `ts-procedures/http`) to determine file types.
-3. Check each file against the categorized checklist in `checklist.md`.
-4. Output findings grouped by severity.
-
-## Output Format
-
-For each finding:
-
-```
-[SEVERITY] file:line — Violation
-  Problem: What's wrong
-  Fix: Concrete before/after code
-```
-
-Severity levels:
-- **CRITICAL** — Will cause bugs, silent failures, resource leaks, or runtime errors. Must fix.
-- **WARNING** — Anti-pattern that hurts maintainability or correctness. Should fix.
-- **SUGGESTION** — Improvement for readability, type safety, or DX. Nice to have.
-
-## Summary
-
-After individual findings, provide:
-- Total findings by severity
-- Overall assessment (healthy / needs attention / significant issues)
-- Top 3 priorities to address
-
-## Reference
-
-See `checklist.md` for the complete categorized checklist by file type.