Skip to content

chore: remove debug console statements from js/EnsembleBlocks.js #5748

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
omsuneri merged 2 commits into from
Feb 16, 2026
Merged

chore: remove debug console statements from js/EnsembleBlocks.js #5748

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3bf0654
chore: remove debug console statements from js/EnsembleBlocks.js
subhraneel2005 Feb 15, 2026
b8ec8cf
fix: eslint formatting
subhraneel2005 Feb 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 9 additions & 22 deletions js/blocks/EnsembleBlocks.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,22 +38,19 @@ function getTargetTurtle(turtles, targetTurtle) {
}
}

// eslint-disable-next-line no-console
console.debug(`turtle "${targetTurtle}" not found`);
//Debug: `turtle "${targetTurtle}" not found` returning null
return null;
}

function _blockFindTurtle(activity, turtle, blk, receivedArg) {
const cblk = activity.blocks.blockList[blk].connections[1];
if (cblk === null) {
// eslint-disable-next-line no-console
console.debug("Could not find connecting block");
//Debug: connecting block not found, returning null
return null;
}
const targetTurtle = activity.logo.parseArg(activity.logo, turtle, cblk, blk, receivedArg);
if (targetTurtle === null) {
// eslint-disable-next-line no-console
console.debug("Could not find target turtle name from arg");
//Debug: target turtleName from arg not found, returning null
return null;
}
return activity.turtles.getTurtle(getTargetTurtle(activity.turtles, targetTurtle));
Expand Down Expand Up @@ -213,8 +210,7 @@ function setupEnsembleBlocks(activity) {
tur.parentFlowQueue = [];
tur.unhighlightQueue = [];
tur.parameterQueue = [];
// eslint-disable-next-line no-console
console.debug("stopping " + targetTurtle);

activity.turtles.ithTurtle(turtle).running = false;
logo.doBreak(tur);
}
Expand Down Expand Up @@ -255,8 +251,6 @@ function setupEnsembleBlocks(activity) {
}

flow(args, logo, turtle, blk, receivedArg, actionArgs, isflow) {
// eslint-disable-next-line no-console
console.debug("start mouse from mouse " + turtle);
if (args[0] === null) {
activity.errorMsg(NOINPUTERRORMSG, blk);
return;
Expand Down Expand Up @@ -296,8 +290,6 @@ function setupEnsembleBlocks(activity) {
}
}
if (foundStartBlock) {
// eslint-disable-next-line no-console
console.debug("STARTING " + targetTurtle + " " + startBlk);
logo.runFromBlock(logo, targetTurtle, startBlk, isflow, receivedArg);
} else {
activity.errorMsg(_("Cannot find start block") + " " + args[0], blk);
Expand Down Expand Up @@ -538,8 +530,7 @@ function setupEnsembleBlocks(activity) {

if (thisTurtle) return activity.turtles.screenY2turtleY(thisTurtle.container.y);
thisTurtle = activity.turtles.getTurtle(turtle);
// eslint-disable-next-line no-console
console.debug("Mouse not found. Using current mouse value instead.");
// Debug: Mouse not found. Using current mouse value instead (fallback behavior)
return activity.turtles.screenY2turtleY(thisTurtle.container.y);
}
}
Expand Down Expand Up @@ -585,8 +576,7 @@ function setupEnsembleBlocks(activity) {
if (thisTurtle) return activity.turtles.screenX2turtleX(thisTurtle.container.x);

thisTurtle = activity.turtles.getTurtle(turtle);
// eslint-disable-next-line no-console
console.debug("Mouse not found. Using current mouse value instead.");
// Debug: Mouse not found. Using current mouse value instead (fallback behavior)
return activity.turtles.screenX2turtleX(thisTurtle.container.x);
}
}
Expand Down Expand Up @@ -716,8 +706,7 @@ function setupEnsembleBlocks(activity) {
logo.synth.inTemperament
);
} else {
// eslint-disable-next-line no-console
console.debug("Cannot find a note for mouse " + turtle);
//Debug: note for mouse not found + turtle
activity.errorMsg(INVALIDPITCH, blk);
obj = ["G", 4];
}
Expand Down Expand Up @@ -755,8 +744,7 @@ function setupEnsembleBlocks(activity) {
logo.synth.inTemperament
);
} else {
// eslint-disable-next-line no-console
console.debug("Cannot find a note for mouse " + turtle);
//Debug: note for mouse not found + turtle
activity.errorMsg(INVALIDPITCH, blk);
obj = ["G", 4];
}
Expand Down Expand Up @@ -1030,8 +1018,7 @@ function setupEnsembleBlocks(activity) {

activity.blocks.loadNewBlocks(newBlock);
} else {
// eslint-disable-next-line no-console
console.debug("Turtle " + turtleName + " already exists.");
// Debug: Attempted to create turtle that already exists
activity.stage.dispatchEvent(turtleName);
}
}
Expand Down
Loading