fix(helm): make lint:helm run

Unrelated to the egress work, but broken in a way worth a one-line fix while the
chart is already in the diff.

`lint:helm` pointed at `helm/sim/test/values-lint.yaml`, which has never existed
in this repository — the script has been dead since the chart landed in #813.
Nobody hit it because CI lints the chart directly rather than through the script.

The reason it was never noticed is the other half: `.gitignore` ignored
`helm/sim/test`, so whoever wrote the script had that file locally and it worked
only on their machine. That entry is removed too. It was also a trap, sitting one
letter away from `helm/sim/tests`, the real committed helm-unittest directory —
anyone who created the singular path would have had it silently ignored.

The script now runs exactly what the workflow runs, so a developer checking
locally gets the same answer as the gate. `--strict` is dropped for the same
reason: a local check stricter than CI reports failures the gate will not.

Author: waleedlatif1

.gitignore

@@ -84,9 +84,6 @@ start-collector.sh
 # IntelliJ
 .idea
 
-## Helm Chart Tests
-helm/sim/test
-
 ## Claude Code
 .claude/launch.json
 .claude/worktrees/

package.json

@@ -27,7 +27,7 @@
     "format:check": "turbo run format:check",
     "lint": "turbo run lint",
     "lint:check": "turbo run lint:check",
-    "lint:helm": "helm lint ./helm/sim --strict --values ./helm/sim/test/values-lint.yaml",
+    "lint:helm": "helm lint helm/sim --values helm/sim/ci/default-values.yaml",
     "lint:all": "turbo run lint && bun run lint:helm",
     "check": "turbo run format:check",
     "check:egress-boundary": "bun run scripts/check-egress-boundary.ts",