Skip to content

Perf: cache condition parsing, type hints, and optimize string operations#452

Merged
thomaspatzke merged 5 commits intomainfrom
copilot/profile-code-performance
Apr 7, 2026
Merged

Perf: cache condition parsing, type hints, and optimize string operations#452
thomaspatzke merged 5 commits intomainfrom
copilot/profile-code-performance

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 6, 2026

  • Identify mypy type checking errors (3 new errors introduced by performance PR)
  • Fix sigma/types.py:586 and :596 - guard escape_char with is not None check before appending to result list
  • Fix sigma/types.py:1030 - use isinstance(v, bool) instead of type(v) is bool for proper type narrowing
  • Fix stale vtype reference after refactoring bool check
  • Run mypy to confirm errors are fixed (back to 3 pre-existing errors, same as main)
  • Run tests to confirm no regressions (1369 passed)
  • Run parallel validation (CodeQL clean, code review addressed)
  • Fix docstring per code review feedback

Copilot AI and others added 2 commits April 5, 2026 22:09
…e string operations

Agent-Logs-Url: https://github.com/SigmaHQ/pySigma/sessions/96b970a7-15bc-4b53-bbeb-51ca1c790721

Co-authored-by: thomaspatzke <1845601+thomaspatzke@users.noreply.github.com>
Copilot AI and others added 2 commits April 6, 2026 23:12
Agent-Logs-Url: https://github.com/SigmaHQ/pySigma/sessions/29062a21-3d71-4f22-b83a-15c994c5ead9

Co-authored-by: thomaspatzke <1845601+thomaspatzke@users.noreply.github.com>
…sibility

Agent-Logs-Url: https://github.com/SigmaHQ/pySigma/sessions/29062a21-3d71-4f22-b83a-15c994c5ead9

Co-authored-by: thomaspatzke <1845601+thomaspatzke@users.noreply.github.com>
@thomaspatzke thomaspatzke merged commit 5b7b7e3 into main Apr 7, 2026
20 checks passed
@thomaspatzke thomaspatzke deleted the copilot/profile-code-performance branch April 7, 2026 22:14
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