Skip to content

Chore(nessusExport): clean up and refactor nessus export - #245

Open
kevjablonski wants to merge 3 commits into
mainfrom
chore/clean-up-and-refactor-nessus-export-233
Open

Chore(nessusExport): clean up and refactor nessus export#245
kevjablonski wants to merge 3 commits into
mainfrom
chore/clean-up-and-refactor-nessus-export-233

Conversation

@kevjablonski

Copy link
Copy Markdown
Collaborator

Summary

Refactors nessusExport.ts to improve type safety, validation, and maintainability while preserving existing Nessus export behavior.

Changes

Changes

  • Replaced unnecessary any casts with existing model types where possible.
  • Added guards for missing Nessus plugin, plugin output, and system data.
  • Added a concrete NessusSoftwareRow return type for generateNessusSW.
  • Simplified Nessus software row construction and removed the array/header size mismatch.
  • Centralized status override normalization and comparison logic.
  • Ensured strict equality checks are used throughout the export logic.

Testing

Verified Nessus CSV export behavior remains unchanged.

Related

Closes #233

Signed-off-by: Kevin Jablonski <kevjablonski@gmail.com>
Signed-off-by: Kevin Jablonski <kevjablonski@gmail.com>
@kevjablonski kevjablonski linked an issue Aug 12, 2026 that may be closed by this pull request
@Amndeep7
Amndeep7 temporarily deployed to tir-chore-clean-up-and--necwkm August 12, 2026 21:36 Inactive
Comment thread server/utils/excelExport/nessusExport.ts Fixed
Comment thread server/utils/excelExport/nessusExport.ts Fixed
Comment thread server/utils/excelExport/nessusExport.ts Fixed
Comment thread server/utils/excelExport/nessusExport.ts Fixed
Signed-off-by: Kevin Jablonski <kevjablonski@gmail.com>
@Amndeep7
Amndeep7 temporarily deployed to tir-chore-clean-up-and--necwkm August 13, 2026 18:18 Inactive
@sonarqubecloud

Copy link
Copy Markdown

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.

Clean Up and Refactor Nessus Export

3 participants