Fix lint workflow and issues that were re-introduced (#1254)

* fix(ci): fix lint workflow to match local linting behaviour and fail on warnings

The lint workflow had two critical issues:

1. **Did not fail on warnings**: The workflow used continue-on-error with exit
   code capture via $?, but $? was always 0 because continue-on-error prevents
   failures. Fixed by checking step outcomes in a final verification step.

2. **Different results than local linting**: CI ran linters from repo root while
   local 'pnpm lint' runs per-package scripts from package directories. This
   caused Solhint's import-path-check rule to produce false warnings in CI.
   Fixed by running Solhint from each package directory using 'pnpm -r exec'.

Additional improvements:
- Use git ls-files instead of globs to only lint tracked files (much faster)
- Exclude Ignition/build artifacts from JSON linting (939 → 102 files, 89% reduction)
- Add --no-warn-ignored to ESLint to suppress ignore pattern warnings
- Add --no-error-on-unmatched-pattern to Prettier to handle symlinks gracefully
- Simplify workflow structure with fewer, clearer steps

Also fixed YAML syntax warning in build-test.yml (removed redundant branches filter).

Author: RembrandtK

.github/workflows/build-test.yml

@@ -6,7 +6,6 @@ env:
 
 on:
   pull_request:
-    branches: '*'
   workflow_dispatch:
 
 jobs: