Skip to content

fix: use correct debug logger prefix#13220

Merged
kgryte merged 1 commit into
developfrom
philipp/drift-plot-components-svg-2026-07-01
Jul 4, 2026
Merged

fix: use correct debug logger prefix#13220
kgryte merged 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

@kgryte kgryte marked this pull request as ready for review July 4, 2026 11:01
@kgryte kgryte requested a review from a team July 4, 2026 11:01
@stdlib-bot stdlib-bot added the Needs Review A pull request which needs code review. label Jul 4, 2026
@kgryte

kgryte commented Jul 4, 2026

Copy link
Copy Markdown
Member

As the CI failure is not due to changes introduced in this PR, I'll go ahead and merge.

@kgryte kgryte changed the title fix: correct debug logger prefix in plot/components/svg/graph fix: use correct debug logger prefix Jul 4, 2026
@kgryte kgryte merged commit 14e698b into develop Jul 4, 2026
34 of 35 checks passed
@kgryte kgryte deleted the philipp/drift-plot-components-svg-2026-07-01 branch July 4, 2026 11:02
@stdlib-bot stdlib-bot removed the Needs Review A pull request which needs code review. label Jul 4, 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.

4 participants