Skip to content

JS-1804 Track RSPEC coverage markers#7131

Open
zglicz wants to merge 3 commits into
masterfrom
codex/rspec-marker-metadata
Open

JS-1804 Track RSPEC coverage markers#7131
zglicz wants to merge 3 commits into
masterfrom
codex/rspec-marker-metadata

Conversation

@zglicz
Copy link
Copy Markdown
Contributor

@zglicz zglicz commented May 29, 2026

Summary

  • add tracked lightweight RSPEC marker metadata for JS/TS and CSS under rspec-rule-markers
  • add marker generation/check scripts and run marker generation from the pre-commit hook
  • keep generated full RSPEC metadata in the existing ignored Maven resource locations so packaged jars still contain full rule data

Verification

  • npm run check-rspec-rule-markers
  • npx prettier --check --ignore-unknown .husky/pre-commit package.json tools/generate-rspec-rule-markers.mjs rspec-rule-markers
  • npm run sync-rspec-all
  • mvn install
  • verified current rspec-tools coverage.all_implemented_rules against a staged-index fresh-checkout export: non-empty buckets js=482, ts=499, CSS=30; S4328 is TS-only; S125 is CSS
  • verified built jars contain full generated metadata: javascript-checks has 529 rule JSON files, css has 30 rule JSON files

@zglicz zglicz temporarily deployed to sca-checking May 29, 2026 12:30 — with GitHub Actions Inactive
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Track RSPEC coverage markers JS-1804 Track RSPEC coverage markers May 29, 2026
@hashicorp-vault-sonar-prod
Copy link
Copy Markdown

hashicorp-vault-sonar-prod Bot commented May 29, 2026

JS-1804

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2026

Ruling Report

No changes to ruling expected issues in this PR

@zglicz zglicz temporarily deployed to sca-checking May 29, 2026 13:53 — with GitHub Actions Inactive
@zglicz zglicz temporarily deployed to sca-checking May 29, 2026 14:13 — with GitHub Actions Inactive
@sonarqube-next
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@gitar-bot
Copy link
Copy Markdown

gitar-bot Bot commented May 29, 2026

Code Review ✅ Approved

Integrates RSPEC marker metadata tracking for JS, TS, and CSS to maintain rule coverage records. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant