dict_init_mutate.py 1.7 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354
  1. # Licensed under the GPL: https://www.gnu.org/licenses/old-licenses/gpl-2.0.html
  2. # For details: https://github.com/pylint-dev/pylint/blob/main/LICENSE
  3. # Copyright (c) https://github.com/pylint-dev/pylint/blob/main/CONTRIBUTORS.txt
  4. """Check for use of dictionary mutation after initialization."""
  5. from __future__ import annotations
  6. from typing import TYPE_CHECKING
  7. from astroid import nodes
  8. from pylint.checkers import BaseChecker
  9. from pylint.checkers.utils import only_required_for_messages
  10. from pylint.interfaces import HIGH
  11. if TYPE_CHECKING:
  12. from pylint.lint.pylinter import PyLinter
  13. class DictInitMutateChecker(BaseChecker):
  14. name = "dict-init-mutate"
  15. msgs = {
  16. "C3401": (
  17. "Declare all known key/values when initializing the dictionary.",
  18. "dict-init-mutate",
  19. "Dictionaries can be initialized with a single statement "
  20. "using dictionary literal syntax.",
  21. )
  22. }
  23. @only_required_for_messages("dict-init-mutate")
  24. def visit_assign(self, node: nodes.Assign) -> None:
  25. """
  26. Detect dictionary mutation immediately after initialization.
  27. At this time, detecting nested mutation is not supported.
  28. """
  29. match node:
  30. case nodes.Assign(
  31. targets=[nodes.AssignName(name=dict_name)], value=nodes.Dict()
  32. ):
  33. pass
  34. case _:
  35. return
  36. match node.next_sibling():
  37. case nodes.Assign(
  38. targets=[nodes.Subscript(value=nodes.Name(name=name))]
  39. ) if (name == dict_name):
  40. self.add_message("dict-init-mutate", node=node, confidence=HIGH)
  41. def register(linter: PyLinter) -> None:
  42. linter.register_checker(DictInitMutateChecker(linter))