Enable Deepnote snapshots by default

[Artmann]

Summary

Changes the default value of the deepnote.snapshots.enabled configuration setting from false to true, enabling snapshot functionality for Deepnote notebooks by default.

Changes

• Updated the default value in package.json configuration schema from false to true
• Updated default parameter in DeepnoteActivationService.isSnapshotsEnabled() method
• Updated default parameter in SnapshotService.isSnapshotsEnabled() method
• Updated unit tests to reflect the new default value in mock configurations

Details

This change makes snapshot functionality opt-out rather than opt-in, meaning outputs will now be saved to separate snapshot files in a 'snapshots' folder by default instead of being stored in the main .deepnote file. Users who prefer the previous behavior can disable this feature by setting deepnote.snapshots.enabled to false in their workspace configuration.

All related code paths and tests have been updated to maintain consistency across the codebase.

https://claude.ai/code/session_012kUL7YuMgYq3CR1xqnrLb5

Summary by CodeRabbit

• Chores

  • Configuration change: Deepnote snapshot outputs are now enabled by default across all notebooks, changing from disabled by default to provide automatic snapshot functionality without requiring users to manually enable the feature.

• Tests

  • Updated unit tests to reflect the new default snapshot configuration.

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request updates the default configuration value for deepnote.snapshots.enabled from false to true across the codebase. Changes include: modifying the default in package.json, updating the configuration reader in deepnoteActivationService.ts to use true as the default, changing the snapshot service logic in snapshotService.ts to default to enabled, and adjusting the corresponding unit tests to validate the new default behavior.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely summarizes the main change: enabling Deepnote snapshots by default. All modifications across multiple files serve this objective.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[dinohamzic]

YES! ❤️

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0%. Comparing base (07855da) to head (076d635).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@     Coverage Diff     @@
##   main   #313   +/-   ##
===========================
===========================

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.