8000 feat(@deepnote/mcp): implement tool behaviors for enhance, fix circul… by jamesbhobbs · Pull Request #258 · deepnote/deepnote · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@jamesbhobbs
Copy link
Contributor

…ar deps, and refactor

enhance tool:

  • Now actually applies input blocks (input-slider) for hardcoded values like test_size, n_estimators, threshold, learning_rate, max_depth
  • Modifies code to reference the input variable instead of hardcoded value
  • Adds section headers (h1, h2) when structure enhancement is applied

fix tool:

  • Implements circular dependency detection using DAG analysis
  • Attempts to break cycles by reordering blocks (moving high-output blocks earlier)
  • Reports detected cycles with the variables involved

refactor tool:

  • Now removes extracted functions/classes from source blocks
  • Adds import statements to source notebooks (from import )
  • Reports sourceModified and importAdded in response

Updated tool descriptions to accurately reflect new capabilities.

jamesbhobbs and others added 2 commits January 29, 2026 12:49
…ar deps, and refactor

enhance tool:
- Now actually applies input blocks (input-slider) for hardcoded values like
  test_size, n_estimators, threshold, learning_rate, max_depth
- Modifies code to reference the input variable instead of hardcoded value
- Adds section headers (h1, h2) when structure enhancement is applied

fix tool:
- Implements circular dependency detection using DAG analysis
- Attempts to break cycles by reordering blocks (moving high-output blocks earlier)
- Reports detected cycles with the variables involved

refactor tool:
- Now removes extracted functions/classes from source blocks
- Adds import statements to source notebooks (from <module> import <names>)
- Reports sourceModified and importAdded in response

Updated tool descriptions to accurately reflect new capabilities.
@coderabbitai
Copy link
Contributor
coderabbitai bot commented Jan 29, 2026

Caution

Review failed

The head commit changed during the review from 81c9fb4 to 0108e64.


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

@jamesbhobbs jamesbhobbs marked this pull request as ready for review January 29, 2026 12:59
@jamesbhobbs jamesbhobbs merged commit f2be05f into mcp-skills Jan 29, 2026
18 checks passed
@jamesbhobbs jamesbhobbs deleted the mcp2 branch January 29, 2026 12:59
@codecov
Copy link
codecov bot commented Jan 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.81%. Comparing base (bf6c4d2) to head (0108e64).
⚠️ Report is 4 commits behind head on mcp-skills.

Additional details and impacted files
@@             Coverage Diff             @@
##           mcp-skills     #258   +/-   ##
===========================================
  Coverage       72.81%   72.81%           
===========================================
  Files              96       96           
  Lines            6909     6909           
  Branches         1977     1918   -59     
===========================================
  Hits             5031     5031           
  Misses           1877     1877           
  Partials            1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

0