Skip to content

fix(detector): detect ModuleType dict replay#137

Closed
prasannakotyal wants to merge 1 commit into
gpu-mode:mainfrom
prasannakotyal:kg-blue-318-module-dict-replay
Closed

fix(detector): detect ModuleType dict replay#137
prasannakotyal wants to merge 1 commit into
gpu-mode:mainfrom
prasannakotyal:kg-blue-318-module-dict-replay

Conversation

@prasannakotyal

Copy link
Copy Markdown

Summary

Targets KG red #318, where custom_kernel aliases a module object's __dict__, stores the first input-derived result through that alias, then returns the alias subscript from an input-independent fast path.

This adds a narrow MODULE_DICT_REPLAY detector. It requires:

  • a module-level ModuleType object
  • an entrypoint local alias assigned from module.__dict__
  • an input-derived assignment to a subscript of that alias
  • an input-independent guard over the same alias
  • a call-free replay body returning a subscript of that alias

Module attribute replay, exec-defined module entrypoints, and generic vars(...) boxes stay out of scope.

KernelGuard-Red-Submission: 318

Local verification

  • red_144_module_dict_replay -> MODULE_DICT_REPLAY, should_filter=True
  • red_109_module_attr_replay stayed valid
  • red_376_module_exec_attr_define stayed valid
  • red_56_vars_box_replay stayed valid
  • clean fixtures in eval_blue_patch.py stayed valid
  • uv run python -m py_compile kernelguard.py

@prasannakotyal prasannakotyal temporarily deployed to kernelguard-api-control-plane May 2, 2026 00:09 — with GitHub Actions Inactive
@github-actions

github-actions Bot commented May 2, 2026

Copy link
Copy Markdown

KernelGuard Blue Evaluation

@SinatrasC

Copy link
Copy Markdown
Collaborator

Thanks for the KernelGuard Flywheel Campaign contribution. We are not merging this narrow variant separately because the consolidated rule-family implementation in #273 is the merge path for this detector area.

@SinatrasC SinatrasC closed this Jun 20, 2026
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