Skip to content

fix: SEO improvements, CI hardening, and lint cleanup#132

Merged
jaysin586 merged 5 commits into
mainfrom
fix/ci-seo-lint-cleanup
Mar 23, 2026
Merged

fix: SEO improvements, CI hardening, and lint cleanup#132
jaysin586 merged 5 commits into
mainfrom
fix/ci-seo-lint-cleanup

Conversation

@jaysin586

Copy link
Copy Markdown
Contributor

Summary

  • CI: Scope secrets to GitHub Environments (ci / production) to resolve zizmor secrets-outside-env warnings
  • SEO: Trim all 17 meta descriptions exceeding 160 characters (range was 162–185) to resolve Ahrefs Site Audit warnings
  • Lint: Remove unnecessary trunk-ignore(eslint/@typescript-eslint/no-explicit-any) comments in LRU tests (rule not active in test files)
  • Trunk: Add osv-scanner ignore for pnpm-lock.yaml to match other Humanspeak repos
  • Deps: Bump dependencies and trunk plugins

Test plan

  • Verify CI workflows pass with new environment: declarations
  • Confirm meta descriptions render correctly on docs site
  • Run trunk check to verify no remaining lint issues

🤖 Generated with Claude Code

jaysin586 and others added 5 commits March 23, 2026 07:07
Add `environment: ci` to debug-check and build jobs, and
`environment: production` to publish-github-packages. Resolves
zizmor secrets-outside-env warnings.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Shorten all 17 meta descriptions that exceeded 160 characters
(range was 162-185). Resolves Ahrefs Site Audit warning.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The no-explicit-any rule is not active in test files, so the
trunk-ignore directives were flagged as doing nothing.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jaysin586 jaysin586 self-assigned this Mar 23, 2026
@jaysin586 jaysin586 added documentation Improvements or additions to documentation skip-publish This is something important, but we dont want to publish it labels Mar 23, 2026
@jaysin586 jaysin586 merged commit 5b94cc6 into main Mar 23, 2026
6 checks passed
@jaysin586

Copy link
Copy Markdown
Contributor Author

⏭️ NPM publishing was skipped due to the skip-publish label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation skip-publish This is something important, but we dont want to publish it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant