fix: CI pipeline failures — lockfile, lint exclusions, Go check

* fix: CI failures — commit package-lock.json and exclude test-all.mjs from lint

- Remove package-lock.json from .gitignore so npm ci works in CI
- Commit the lockfile for reproducible builds
- Exclude test-all.mjs from hardcoded path check (it legitimately
  contains /Users/ patterns as comments and grep assertions)

Fixes all 7 failing CI jobs (6 test matrix + lint).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: exclude PR template and ci.yml from hardcoded path checks

Both files legitimately contain /Users/ as example text in
checklists and grep patterns. Exclude them from both the CI
lint step and the test-all.mjs absolute path check.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: skip dashboard build test when Go is not installed

macOS CI runners don't have Go pre-installed. The dashboard build
is already tested in a dedicated CI job with actions/setup-go.
The test suite now gracefully warns instead of failing.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: RajjjAryan

.github/workflows/ci.yml

@@ -66,7 +66,8 @@ jobs:
             --include='*.mjs' --include='*.js' --include='*.go' \
             --include='*.md' --include='*.yml' --include='*.yaml' \
             --exclude-dir=node_modules --exclude-dir=.git \
-            --exclude='*.lock' .; then
+            --exclude='*.lock' --exclude='test-all.mjs' \
+            --exclude='PULL_REQUEST_TEMPLATE.md' --exclude='ci.yml' .; then
             echo "::error::Found hardcoded user paths in source files"
             exit 1
           fi

.gitignore

@@ -36,7 +36,6 @@ article-digest.md
 # Generated
 .resolved-prompt-*
 career-dashboard
-package-lock.json
 
 # OS
 .DS_Store

package-lock.json

@@ -83,11 +83,16 @@ for (const { name, allowFail } of scripts) {
 
 if (!QUICK) {
   console.log('\n3. Dashboard build');
-  const goBuild = run('cd dashboard && go build -o /tmp/career-dashboard-test . 2>&1');
-  if (goBuild !== null) {
-    pass('Dashboard compiles');
+  const goAvailable = run('which go 2>/dev/null');
+  if (!goAvailable) {
+    warn('Go not installed — skipping dashboard build');
   } else {
-    fail('Dashboard build failed');
+    const goBuild = run('cd dashboard && go build -o /tmp/career-dashboard-test . 2>&1');
+    if (goBuild !== null) {
+      pass('Dashboard compiles');
+    } else {
+      fail('Dashboard build failed');
+    }
   }
 } else {
   console.log('\n3. Dashboard build (skipped --quick)');
@@ -211,7 +216,7 @@ if (!leakFound) {
 console.log('\n6. Absolute path check');
 
 const absPathResult = run(
-  `grep -rn "/Users/" --include="*.mjs" --include="*.sh" --include="*.md" --include="*.go" --include="*.yml" . 2>/dev/null | grep -v node_modules | grep -v ".git/" | grep -v README.md | grep -v LICENSE | grep -v go.sum | grep -v copilot-instructions.md | grep -v test-all.mjs`
+  `grep -rn "/Users/" --include="*.mjs" --include="*.sh" --include="*.md" --include="*.go" --include="*.yml" . 2>/dev/null | grep -v node_modules | grep -v ".git/" | grep -v README.md | grep -v LICENSE | grep -v go.sum | grep -v copilot-instructions.md | grep -v test-all.mjs | grep -v PULL_REQUEST_TEMPLATE | grep -v ci.yml`
 );
 if (!absPathResult) {
   pass('No absolute paths in code files');