Merge pull request #24 from shift-editor/kostya/deslop

kostya/deslop

Author: kostyafarber

.claude/settings.json

@@ -0,0 +1,15 @@
+{
+  "hooks": {
+    "Stop": [
+      {
+        "matcher": "",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "git diff --name-only --diff-filter=ACMR HEAD | grep -E '\\.(ts|tsx|json|md|css)$' | xargs pnpm prettier --write --ignore-unknown 2>/dev/null; true"
+          }
+        ]
+      }
+    ]
+  }
+}

.claude/skills/add-manager/SKILL.md

@@ -99,7 +99,7 @@ jobs:
 
       - run: pnpm install --frozen-lockfile --prefer-offline
       - name: Typecheck
-        run: pnpm test:typecheck
+        run: pnpm typecheck
 
   unit-test:
     name: Unit Tests
@@ -120,6 +120,10 @@ jobs:
           cache: pnpm
 
       - run: pnpm install --frozen-lockfile --prefer-offline
+
+      - name: Build native module
+        run: pnpm build:native
+
       - name: Run unit tests
         run: pnpm test:unit
 
@@ -191,7 +195,7 @@ jobs:
         run: cargo clippy --workspace --all-targets -- -D warnings
 
       - name: Run tests
-        run: cargo test --workspace
+        run: cargo test --workspace --exclude shift-node
 
   integration:
     name: Integration Tests

.claude/skills/geometry-boundaries/SKILL.md

@@ -9,9 +9,38 @@ repos:
       - id: check-added-large-files
       - id: check-merge-conflict
 
+  # Slop detection
+  - repo: local
+    hooks:
+      - id: no-empty-files
+        name: no empty files
+        entry: "bash -c 'for f in \"$@\"; do if [ -f \"$f\" ] && [ ! -s \"$f\" ]; then echo \"Empty file: $f\"; exit 1; fi; done'"
+        language: system
+        types_or: [javascript, jsx, ts, tsx, rust, css, json]
+
+      - id: no-console-log
+        name: no console.log in production code
+        entry: "bash -c 'for f in \"$@\"; do case \"$f\" in *.test.*|*/testing/*) continue;; esac; if grep -n \"^\\s*console\\.log\\b\" \"$f\" >/dev/null 2>&1; then echo \"$f has console.log:\"; grep -n \"^\\s*console\\.log\\b\" \"$f\"; exit 1; fi; done'"
+        language: system
+        types_or: [javascript, jsx, ts, tsx]
+
   # Rust hooks
   - repo: local
     hooks:
+      - id: napi-dead-methods
+        name: napi dead methods
+        entry: ./scripts/check-napi-dead-methods.sh
+        language: system
+        types: [rust]
+        pass_filenames: false
+
+      - id: cargo-check-tests
+        name: cargo check (including tests)
+        entry: bash -c 'cargo check --package shift-node --tests'
+        language: system
+        types: [rust]
+        pass_filenames: false
+
       - id: cargo-fmt
         name: cargo fmt
         entry: cargo fmt --all --