Skip to content

fix: correct debug logger prefix in plot/components/svg/graph#13220

Draft
Planeshifter wants to merge 1 commit into
developfrom
philipp/drift-plot-components-svg-2026-07-01
Draft

fix: correct debug logger prefix in plot/components/svg/graph#13220
Planeshifter wants to merge 1 commit into
developfrom
philipp/drift-plot-components-svg-2026-07-01

Conversation

@Planeshifter

Copy link
Copy Markdown
Member

Description

This pull request:

  • Corrects a copy-paste bug in @stdlib/plot/components/svg/graph/lib/props/auto-render/set.js. The debug logger was initialized with the prefix 'canvas:set:auto-render' — carried over from the sibling canvas package — instead of 'graph:set:auto-render'. As a consequence, debug output from the graph component's auto-render setter was emitted under the wrong namespace and would not surface when a user set DEBUG=graph:*.

plot/components/svg/graph

Change the debug logger prefix in lib/props/auto-render/set.js from 'canvas:set:auto-render' to 'graph:set:auto-render'. Every other logger in the graph package already uses the graph:* prefix, and 7 of 8 sibling packages in @stdlib/plot/components/svg/* that expose an auto-render setter prefix their logger with their own package slug (~87% conformance). Behavior is unchanged; only the debug label is corrected.

Related Issues

No.

Questions

No.

Other

Diff is a one-line string change under lib/props/auto-render/set.js; no source, test, or documentation depends on the old string.

Checklist

AI Assistance

  • Yes
  • No

If you answered "yes" above, how did you use AI assistance?

  • Code generation (e.g., when writing an implementation or fixing a bug)
  • Test/benchmark generation
  • Documentation (including examples)
  • Research and understanding

Disclosure

This PR was authored by an automated drift-detection routine run under Claude Code. The routine sampled the @stdlib/plot/components/svg namespace, extracted structural and semantic features from all 12 packages, identified the debug-logger-prefix convention as a majority pattern with a single deviating package, and applied a mechanical one-line fix. Three independent validation agents confirmed the deviation as a copy-paste artifact with no dependents.



Generated by Claude Code

The `lib/props/auto-render/set.js` file initialized its `debug` logger
with the prefix `'canvas:set:auto-render'`, a copy-paste artifact from
the sibling `canvas` package. Every other debug logger in the `graph`
package uses the `graph:*` prefix, and 7 of the 8 sibling packages in
`@stdlib/plot/components/svg/*` that expose an `auto-render` setter
prefix their logger with their own package name (majority conformance
~87%). Change the string to `'graph:set:auto-render'` so debug output
is filtered under `DEBUG=graph:*` as expected.

Copy link
Copy Markdown
Member Author

Note on the failing run_tests_coverage_pr check: the @stdlib/plot/components/svg/graph package has no test/ directory (a pre-existing structural gap shared with 8 of the 12 plot component packages in this namespace), so the coverage job has no tests to run against the changed file. The change itself is a one-line copy-paste fix to a debug logger string with no observable behavior impact; adding a test suite for the untested graph constructor is out of scope for this drift correction.


Generated by Claude Code

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