Skip to content

Comments

feat(modules): builtin nuclei scan as module#61

Merged
vmfunc merged 7 commits intovmfunc:mainfrom
0x4bs3nt:feat/builtin-nuclei
Jan 12, 2026
Merged

feat(modules): builtin nuclei scan as module#61
vmfunc merged 7 commits intovmfunc:mainfrom
0x4bs3nt:feat/builtin-nuclei

Conversation

@0x4bs3nt
Copy link
Contributor

@0x4bs3nt 0x4bs3nt commented Jan 7, 2026

First part of completing the #52 issue by making requested complex Go scans available as modules.

Summary

  • Allows complex Go scan; nuclei, to participate in the sif module system

Features

  • Made Go scan for nuclei available as a module
  • Added to module registry
  • Will show up in the -lm/-list-modules

Test plan

  • gofmt applied
  • all unit tests passing
  • go vet passes
  • manually tested nuclei works the same as a module and as a legacy Go scan

Infrastructure preparation for builtin complex Go module registration.
Converted nuclei scan to be able to run as module.
@0x4bs3nt 0x4bs3nt changed the title feat/builtin-nuclei feat(modules): builtin nuclei scan as module Jan 7, 2026
@0x4bs3nt 0x4bs3nt marked this pull request as draft January 7, 2026 13:12
Fixed nil pointer dereference issues in the nuclei scan running as a
module
@0x4bs3nt 0x4bs3nt marked this pull request as ready for review January 7, 2026 14:09
@0x4bs3nt 0x4bs3nt marked this pull request as draft January 7, 2026 14:22
Set the HostErrorsCache executor option, cache is created but not passed
as option.
Headless initialization is required even without browser templates.
Nuclei expects project file to be set
@0x4bs3nt 0x4bs3nt marked this pull request as ready for review January 7, 2026 16:52
Fixed Nuclei giving off exception for missing Colorizer on the executor
options.
Copy link
Owner

@vmfunc vmfunc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure having two files named nuclei is the best course of action here

@0x4bs3nt
Copy link
Contributor Author

0x4bs3nt commented Jan 7, 2026

not sure having two files named nuclei is the best course of action here

Yeah, makes sense. Will change on all other PRs as well.

Renamed the nuclei module file to differentiate from the nuclei legacy
scan file.
@0x4bs3nt 0x4bs3nt requested a review from vmfunc January 7, 2026 21:36
@vmfunc vmfunc merged commit c988e54 into vmfunc:main Jan 12, 2026
7 checks passed
@0x4bs3nt 0x4bs3nt deleted the feat/builtin-nuclei branch January 12, 2026 10:55
vmfunc added a commit that referenced this pull request Feb 14, 2026
feat(modules): builtin nuclei scan as module
vmfunc added a commit that referenced this pull request Feb 14, 2026
feat(modules): builtin nuclei scan as module
vmfunc added a commit that referenced this pull request Feb 14, 2026
feat(modules): builtin nuclei scan as module
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.

2 participants