Skip to content

Refactor code structure for improved readability and maintainability - #13

Merged
akanthed merged 1 commit into
mainfrom
fix/ofss-job
Aug 6, 2026
Merged

Refactor code structure for improved readability and maintainability#13
akanthed merged 1 commit into
mainfrom
fix/ofss-job

Conversation

@akanthed

@akanthed akanthed commented Aug 6, 2026

Copy link
Copy Markdown
Owner

What and why

Detection logic changes (delete this section if not applicable)

  • npm run regression was run and every proven/likely finding it printed was reviewed against its source line
  • A new/updated rule has fixtures in both test-fixtures/vulnerable/ and test-fixtures/safe/
  • src/scanner/catalog.ts and src/scanner/explainer.ts both have an entry for any new rule ID
  • test/corpus.test.js's EXPECTED_VULNERABLE was updated for any new rule

CLI changes (delete this section if not applicable)

  • test/cli.test.js has a new or updated case exercising the built binary (src/cli.ts changes aren't covered by any other test file)

Checklist

  • npm run build && npm test passes
  • Before/after behavior is described above, with a concrete example if the change affects scan output

@akanthed
akanthed merged commit b9c60bd into main Aug 6, 2026
11 checks passed
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