Skip to content

Remove debug console.log statements from session restore#3

Draft
Copilot wants to merge 2 commits intofeat/session-save-restorefrom
copilot/sub-pr-1-again
Draft

Remove debug console.log statements from session restore#3
Copilot wants to merge 2 commits intofeat/session-save-restorefrom
copilot/sub-pr-1-again

Conversation

Copy link
Copy Markdown

Copilot AI commented Dec 20, 2025

Addresses review feedback to remove debug logging from session restore functionality before production merge.

Changes:

  • Removed 5 console.log('[SESSION] ...') debug statements from file open and session save flows
  • Replaced console.error with log.error using existing electron-log utility
  • Removed empty .then() callback that only contained debug logging

Example:

// Before
console.log('[SESSION] Saving session with files:', allOpenedFiles)
dataCenter.setItem('lastSession', allOpenedFiles).then(() => {
  console.log('[SESSION] Session saved successfully')
}).catch(err => {
  console.error('[SESSION] Failed to save:', err)
})

// After
dataCenter.setItem('lastSession', allOpenedFiles).catch(err => {
  log.error('[SESSION] Failed to save:', err)
})

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: walidsi <42148514+walidsi@users.noreply.github.com>
Copilot AI changed the title [WIP] Update session restore functionality based on review feedback Remove debug console.log statements from session restore Dec 20, 2025
Copilot AI requested a review from walidsi December 20, 2025 20:43
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