Remove commented out console logs

Author: dwertheimer

dwertheimer.EventAutomations/src/config.js

@@ -84,7 +84,6 @@ export function validateAutoTimeBlockingConfig(config: AutoTimeBlockingConfig):
     // $FlowIgnore
     return validatedConfig
   } catch (error) {
-    // console.log(`NPTimeblocking::validateAutoTimeBlockingConfig: ${String(error)}\nInvalid config:\n${JSON.stringify(config)}`)
     throw new Error(`${String(error)}`)
   }
 }

dwertheimer.EventAutomations/src/timeblocking-helpers.js

@@ -276,7 +276,7 @@ export function findTimeBlocks(timeMap: IntervalMap, config: { [key: string]: an
     let blockStart = timeMap[0]
     for (let i = 1; i < timeMap.length; i++) {
       const slot = timeMap[i]
-      // console.log(`findTimeBlocks[${i}]: slot: ${slot.start} ${slot.index} ${slot.busy}}`)
+
       const noBreakInContinuity = slot.index === lastSlot.index + 1 && i <= timeMap.length - 1 && lastSlot.busy === slot.busy
       if (noBreakInContinuity) {
         lastSlot = slot
@@ -312,9 +312,7 @@ export function findTimeBlocks(timeMap: IntervalMap, config: { [key: string]: an
       if (lastBlock) blocks.push(lastBlock)
     }
   } else {
-    // console.log(`findTimeBlocks: timeMap array was empty`)
   }
-  // console.log(`findTimeBlocks: found blocks: ${JSP(blocks)}`)
 
   return blocks
 }

dwertheimer.MathSolver/src/support/solver.js

@@ -298,7 +298,6 @@ export function parse(thisLineStr: string, lineIndex: number, cd: CurrentData):
   // SOURCE: https://stackoverflow.com/questions/12812902/javascript-regular-expression-matching-cityname // how to take only specific parts
   const reg = /(\d*[\.,])?(\d+)(\s?%)(\s+)(of)(\s+)(\d*[\.,])?(\d+\s?)/g
   while ((match = reg.exec(strToBeParsed))) {
-    // console.log(match);
     const num = match[1] ? match[1] + match[2] : match[2]
     const dest = match[7] ? match[7] + match[8] : match[8]
     const sostituzione = (Number(dest) * (Number(num) / 100)).toString()

dwertheimer.ReactSkeleton/src/react/support/performRollup.node.js

@@ -38,7 +38,7 @@ const { rollupReactFiles, getRollupConfig } = rollupReactScript
   ]
   // create one single base config with two output options
   const config = { ...rollupConfigs[0], ...{ output: [rollupConfigs[0].output, rollupConfigs[1].output] } }
-  // console.log(JSON.stringify(config, null, 2))
+
   await rollupReactFiles(config, watch, 'dwertheimer.ReactSkeleton: development && production')
   // const rollupsProms = rollups.map((obj) => rollupReactFiles({ ...obj, buildMode }, watch, buildMode))
 })().catch((error) => {

dwertheimer.TaskAutomations/src/react/EditableElement.jsx

@@ -22,7 +22,7 @@ export const EditableElement = (props) => {
     ref: element,
     onKeyUp: onMouseUp,
   })
-  // console.log(`WebView: EditableElement elements=`, elements)
+
   return elements
 }
 

dwertheimer.TaskAutomations/src/react/ThemedSelect.jsx

@@ -196,7 +196,6 @@ const colourStyles = {
   // option: (styles) => ({ ...styles, backgroundColor: NP_THEME.base.backgroundColor, color: NP_THEME.base.textColor ?? 'black' }),
   // option: (styles, { data, isDisabled, isFocused, isSelected }) => {
   option: (styles, { isDisabled, isSelected }) => {
-    // console.log('option', styles, data, isDisabled, isFocused, isSelected)
     return {
       ...styles,
       // backgroundColor: isDisabled ? undefined : isSelected ? bgColor.css() : isFocused ? bgColor.alpha(0.1).css() : bgColor.css(),

dwertheimer.TaskAutomations/src/react/support/performRollup.node.js

@@ -39,7 +39,7 @@ const { rollupReactFiles, getCommandLineOptions, getRollupConfig } = rollupReact
   ]
   // create one single base config with two output options
   const config = { ...rollupConfigs[0], ...{ output: [rollupConfigs[0].output, rollupConfigs[1].output] } }
-  // console.log(JSON.stringify(config, null, 2))
+
   await rollupReactFiles(config, watch, 'TaskAutomations: development && production')
   // const rollupsProms = rollups.map((obj) => rollupReactFiles({ ...obj, buildMode }, watch, buildMode))
 })()

dwertheimer.TaskSorting/__tests__/sortTasks.test.js

@@ -394,7 +394,7 @@ describe(`${PLUGIN_NAME}`, () => {
         // output order is the reverse of that order
         // Note that types will be unreliable because rawContent is being pasted
         // so we're just checking the content
-        // console.log(`sortTasks result`, result)
+
         expect(result[8].content).toEqual('6-checklistCancelled')
         expect(result[7].content).toEqual('5-cancelled')
         expect(result[6].content).toEqual('4-checklistDone')
@@ -579,7 +579,7 @@ describe(`${PLUGIN_NAME}`, () => {
             const shouldBe = `${p.rawContent}`
             const newContent = `${result[i].rawContent}`
             // uncomment the following line if this test is failing and it will give you more clues on how far it got
-            // console.log(`sortTasks: [${i}]: (result) ${newContent} ${newContent === shouldBe ? '===' : ' !== '} "${shouldBe}" (expected)`)
+
             // Put breakpoint on the expect and compare the objects in the debugger
             expect(newContent).toMatch(shouldBe)
           })

helpers/HTMLView.js

@@ -5,9 +5,7 @@
 // Last updated 2025-05-31 by @jgclark
 // ---------------------------------------------------------
 import showdown from 'showdown' // for Markdown -> HTML from https://github.com/showdownjs/showdown
-import {
-  hasFrontMatter
-} from '@helpers/NPFrontMatter'
+import { hasFrontMatter } from '@helpers/NPFrontMatter'
 import { getFolderFromFilename } from '@helpers/folders'
 import { clo, logDebug, logError, logInfo, logWarn, JSP, timer } from '@helpers/dev'
 import { getStoredWindowRect, isHTMLWindowOpen, storeWindowRect } from '@helpers/NPWindows'
@@ -86,7 +84,7 @@ export function getCallbackCodeString(jsFunctionName: string, commandName: strin
             .replace("%%commandName%%", commandName)
             .replace("%%pluginID%%", pluginID)
             .replace("%%commandArgs%%", () => JSON.stringify(commandArgs)); //This is important because it works around problems with $$ in commandArgs
-      // console.log(\`${jsFunctionName}: Sending command "\$\{commandName\}" to NotePlan: "\$\{pluginID\}" with args: \$\{JSON.stringify(commandArgs)\}\`);
+      
       console.log(\`window.${jsFunctionName}: Sending code: "\$\{code\}"\`)
       if (window.webkit) {
         window.webkit.messageHandlers.jsBridge.postMessage({
@@ -101,7 +99,6 @@ export function getCallbackCodeString(jsFunctionName: string, commandName: strin
 `
 }
 
-
 /**
  * Convert a note's content to HTML and include any images as base64
  * @param {string} content
@@ -164,22 +161,22 @@ export async function getNoteContentAsHTML(content: string, note: TNote): Promis
       tasklists: true,
       metadata: false, // otherwise metadata is swallowed
       requireSpaceBeforeHeadingText: true,
-      simpleLineBreaks: true // Makes this GFM style. TODO: make an option?
+      simpleLineBreaks: true, // Makes this GFM style. TODO: make an option?
     }
     const converter = new showdown.Converter(converterOptions)
     let body = converter.makeHtml(lines.join(`\n`))
     body = `<style>img { background: white; max-width: 100%; max-height: 100%; }</style>${body}` // fix for bug in showdown
-    
+
     const imgTagRegex = /<img src=\"(.*?)\"/g
     const matches = [...body.matchAll(imgTagRegex)]
     const noteDirPath = getFolderFromFilename(note.filename)
-    
+
     for (const match of matches) {
       const imagePath = match[1]
       try {
         // Handle both absolute and relative paths
         let fullPath = `../../../Notes/${noteDirPath}/${decodeURI(imagePath)}`
-        if(fullPath.endsWith('.drawing')) {
+        if (fullPath.endsWith('.drawing')) {
           fullPath = fullPath.replace('.drawing', '.png')
         }
         const data = await DataStore.loadData(fullPath, false)
@@ -237,14 +234,12 @@ export async function getNoteContentAsHTML(content: string, note: TNote): Promis
       modifiedLines.push(line)
     }
     return modifiedLines.join('\n')
-
   } catch (error) {
     logError('getNoteContentAsHTML', error.message)
     return '<conversion error>'
   }
 }
 
-
 /**
  * This function creates the webkit console.log/error handler for HTML messages to get back to NP console.log
  * @returns {string} - the javascript (without a tag)
@@ -566,7 +561,10 @@ export async function showHTMLV2(body: string, opts: HtmlWindowOptions): Promise
   try {
     const screenWidth = NotePlan.environment.screenWidth
     const screenHeight = NotePlan.environment.screenHeight
-    logDebug('HTMLView / showHTMLV2', `starting with customId ${opts.customId ?? ''} and reuseUsersWindowRect ${String(opts.reuseUsersWindowRect) ?? '??'} for screen dimensions ${screenWidth}x${screenHeight}`)
+    logDebug(
+      'HTMLView / showHTMLV2',
+      `starting with customId ${opts.customId ?? ''} and reuseUsersWindowRect ${String(opts.reuseUsersWindowRect) ?? '??'} for screen dimensions ${screenWidth}x${screenHeight}`,
+    )
 
     // Assemble the parts of the HTML into a single string
     const fullHTMLStr = assembleHTMLParts(body, opts)
@@ -592,8 +590,8 @@ export async function showHTMLV2(body: string, opts: HtmlWindowOptions): Promise
       winOptions = {
         x: opts.x ?? (screenWidth - (screenWidth - (opts.paddingWidth ?? 0) * 2)) / 2,
         y: opts.y ?? (screenHeight - (screenHeight - (opts.paddingHeight ?? 0) * 2)) / 2,
-        width: opts.width ?? (screenWidth - (opts.paddingWidth ?? 0) * 2),
-        height: opts.height ?? (screenHeight - (opts.paddingHeight ?? 0) * 2),
+        width: opts.width ?? screenWidth - (opts.paddingWidth ?? 0) * 2,
+        height: opts.height ?? screenHeight - (opts.paddingHeight ?? 0) * 2,
         shouldFocus: opts.shouldFocus,
         id: cId, // don't need both ... but trying to work out which is the current one for the API
         windowId: cId,
@@ -603,7 +601,6 @@ export async function showHTMLV2(body: string, opts: HtmlWindowOptions): Promise
         // logDebug('showHTMLV2', `- Trying to use user's saved Rect from pref for ${cId}`)
         const storedRect = getStoredWindowRect(cId)
         if (storedRect) {
-
           winOptions = {
             x: storedRect.x,
             y: storedRect.y,

helpers/NPFrontMatter.js

@@ -346,7 +346,7 @@ export function setFrontMatterVars(note: CoreNoteFields, varObj: { [string]: str
     logDebug(`setFrontMatterVars`, `- BEFORE ensureFM: hasFrontmatter:${String(noteHasFrontMatter(note) || '')} note has ${note.paragraphs.length} lines`)
     const hasFM = ensureFrontmatter(note, true, title)
     logDebug('note.paragraphs', `- AFTER ensureFM has ${note.paragraphs.length} lines, that starts:`)
-    // console.log(note.paragraphs.slice(0, 7).map(p => p.content).join('\n'))
+
     if (!hasFM) {
       throw new Error(`setFrontMatterVars: Could not add front matter to note which has no title. Note should have a title, or you should pass in a title in the varObj.`)
     }
@@ -364,7 +364,6 @@ export function setFrontMatterVars(note: CoreNoteFields, varObj: { [string]: str
       removeFrontMatter(note)
       writeFrontMatter(note, changedAttributes)
       logDebug('setFrontMatterVars', `- ENDING with ${note.paragraphs.length} lines, that starts:`)
-      // console.log(note.paragraphs.slice(0, 7).map(p => p.content).join('\n'))
     } else {
       logError('setFrontMatterVars', `- could not change frontmatter for note "${note.filename || ''}" because it has no frontmatter.`)
     }