From 82545acbaaefee7dca4fe9bc2b61a8eceb5c00e3 Mon Sep 17 00:00:00 2001
From: lucaronin <luca.rossi.917@gmail.com>
Date: Mon, 4 May 2026 00:52:08 +0200
Subject: [PATCH] test: include mobile app in root test runner

---
 .husky/pre-commit       |  2 +-
 docs/MOBILE_PROGRESS.md |  8 +++++++-
 scripts/run-tests.mjs   | 41 +++++++++++++++++++++++++++++++++++++++--
 3 files changed, 47 insertions(+), 4 deletions(-)

diff --git a/.husky/pre-commit b/.husky/pre-commit
index 01fd8f8c..803de433 100755
--- a/.husky/pre-commit
+++ b/.husky/pre-commit
@@ -55,7 +55,7 @@ fi
 
 # Unit tests
 echo "  → tests..."
-pnpm exec vitest run --silent
+pnpm test -- --silent
 
 echo "✅ Pre-commit passed"
 
diff --git a/docs/MOBILE_PROGRESS.md b/docs/MOBILE_PROGRESS.md
index f8a3bfff..701a0fe9 100644
--- a/docs/MOBILE_PROGRESS.md
+++ b/docs/MOBILE_PROGRESS.md
@@ -8,7 +8,7 @@ This file is the resumable working log for Tolaria mobile. The strategy and road
 
 - Branch: `codex/mobile`
 - Active phase: Phase 2 - Mobile Shell
-- Active slice: Define mobile vault repository boundary
+- Active slice: Include mobile tests in root test runner and commit gate
 - Push policy: commit locally; do not push unless explicitly requested
 - Validation target: iPad/iOS simulator first
 
@@ -37,6 +37,8 @@ This file is the resumable working log for Tolaria mobile. The strategy and road
 - Extracted compact phone navigation into a pure tested state machine so future swipe gestures can dispatch explicit events instead of mutating panels ad hoc.
 - Extracted mobile note projection into a pure module that turns raw vault-like note records into the list/editor shape used by the mobile shell.
 - Added a minimal mobile vault repository contract plus fixture implementation so app-local vault storage and git sync can plug in behind `listNotes` / `readNote` later.
+- Updated `scripts/run-tests.mjs` so full `pnpm test` runs desktop, mobile, and shared package tests, while targeted `apps/mobile/...` and `packages/markdown/...` paths route to the correct workspace Vitest config.
+- Updated `.husky/pre-commit` to use `pnpm test -- --silent` so local commits include the new mobile test suite.
 
 ## Next Action
 
@@ -95,6 +97,10 @@ Continue Phase 2 with the next mobile shell slice:
 - `pnpm --filter @tolaria/mobile typecheck` passed after mobile vault repository extraction.
 - CodeScene after mobile vault repository extraction: `apps/mobile/src/mobileVaultRepository.ts` and `apps/mobile/src/mobileVaultRepository.test.ts` scored `10`; `apps/mobile/src/demoData.ts` still returned no scorable code and no findings.
 - `pnpm --filter @tolaria/mobile exec expo export --platform ios --output-dir /tmp/tolaria-mobile-export` passed after mobile vault repository extraction.
+- `pnpm test -- apps/mobile/src/mobileVaultRepository.test.ts` passed and ran only the mobile repository test file.
+- `pnpm test -- packages/markdown/src/noteTitle.test.ts` passed and ran only the shared Markdown note-title test file.
+- `pnpm test -- --silent` passed after runner/hook updates: 309 desktop files / 3639 desktop tests, 4 mobile files / 11 mobile tests, 3 shared Markdown files / 56 shared tests.
+- CodeScene after test-runner update: `scripts/run-tests.mjs` scored `10`; `.husky/pre-commit` is unsupported by CodeScene file analysis because it has no supported extension.
 
 ## Risks / Watch Items
 
diff --git a/scripts/run-tests.mjs b/scripts/run-tests.mjs
index 56467ad1..491e11f3 100644
--- a/scripts/run-tests.mjs
+++ b/scripts/run-tests.mjs
@@ -5,10 +5,26 @@ if (testArgs[0] === '--') {
   testArgs.shift()
 }
 
+const workspaceTarget = findWorkspaceTarget(testArgs)
+if (workspaceTarget) {
+  run('pnpm', [
+    '--filter',
+    workspaceTarget.filter,
+    'exec',
+    'vitest',
+    'run',
+    '--config',
+    'vitest.config.ts',
+    ...workspaceTarget.args,
+  ])
+  process.exit(0)
+}
+
 run('pnpm', ['exec', 'vitest', 'run', ...testArgs])
 
-if (testArgs.length === 0) {
-  run('pnpm', ['--filter', './packages/*', 'test'])
+if (isFullSuite(testArgs)) {
+  run('pnpm', ['--filter', './apps/*', 'test', '--', ...testArgs])
+  run('pnpm', ['--filter', './packages/*', 'test', '--', ...testArgs])
 }
 
 function run(command, args) {
@@ -25,3 +41,24 @@ function run(command, args) {
     process.exit(result.status ?? 1)
   }
 }
+
+function findWorkspaceTarget(args) {
+  const workspacePaths = ['./apps/mobile', './packages/markdown']
+  return workspacePaths.map((workspacePath) => rebaseArgs(workspacePath, args)).find(Boolean)
+}
+
+function rebaseArgs(workspacePath, args) {
+  const prefix = `${workspacePath.slice(2)}/`
+  if (!args.some((arg) => arg.startsWith(prefix))) {
+    return null
+  }
+
+  return {
+    filter: workspacePath,
+    args: args.map((arg) => arg.startsWith(prefix) ? arg.slice(prefix.length) : arg),
+  }
+}
+
+function isFullSuite(args) {
+  return args.every((arg) => arg.startsWith('-'))
+}