src/snakemake/workflow.py (1)

1384-1391: Code is correct, but consider using the formatted display value in the event payload for consistency.

The nodes_str ternary correctly formats the output as "unlimited" when self.nodes == sys.maxsize. However, the extra=dict(..., nodes=self.nodes) still passes the raw integer value to the structured log payload, which will be 9223372036854775807 for unlimited nodes. This inconsistency may confuse downstream log consumers.

Optional: use the formatted value in the event payload

-                        logger.info(
-                            f"Provided remote nodes: {nodes_str}",
-                            extra=dict(event=LogEvent.RESOURCES_INFO, nodes=self.nodes),
-                        )
+                        logger.info(
+                            f"Provided remote nodes: {nodes_str}",
+                            extra=dict(event=LogEvent.RESOURCES_INFO, nodes=nodes_str),
+                        )

Note: This is optional because the current design (numeric value in structured event, formatted string in message) is a valid choice for consumers that need the actual integer value. Only apply this if downstream event consumers expect a human-readable format.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/snakemake/workflow.py` around lines 1384 - 1391, The log message uses
nodes_str for the human-readable message but passes the raw self.nodes integer
in the structured payload; update the logger call so the extra payload uses the
formatted display value (nodes_str) instead of self.nodes to keep message and
event payload consistent—locate the logger.info invocation that currently builds
nodes_str and change extra=dict(event=LogEvent.RESOURCES_INFO, nodes=self.nodes)
to supply nodes=nodes_str (preserving LogEvent.RESOURCES_INFO and the same
logger.info call).