address some of the ruff warnings

[danieleades]

i know the python implementation isn't going to be used in the long-term (i'm assuming that's still the assumption?) but in the meantime, there are errors in main that are causing CI to fail.

Consistently failing jobs means CI feedback on PRs is less meaningful.

i've squashed most of the warnings, except for these-

continuedev/libs/util/copy_codebase.py:104:18: F821 Undefined name `ManualEditAction`
continuedev/plugins/context_providers/embeddings.py:28:29: F821 Undefined name `ChromaIndexManager`
continuedev/plugins/context_providers/embeddings.py:36:34: F821 Undefined name `ChromaIndexManager`

where are these objects defined??

[netlify]

✅ Deploy Preview for continuedev canceled.

Name	Link
🔨 Latest commit	bd5b82a
🔍 Latest deploy log	https://app.netlify.com/sites/continuedev/deploys/65ab81f237791b00087a71a1

[sestinj]

It looks like they are defined...nowhere. ManualEditAction is extremely old code, and I think ChromaIndexManager was a WIP, or just ended up unused.

I've moved the Python server into a separate repo so it might be okay to leave these few missing parts.

[sestinj]

I'm happy to merge this, but it would get cleared when preview merges to main. Do you want to make a separate PR on the other repo?

[danieleades]

I'm happy to merge this, but it would get cleared when preview merges to main. Do you want to make a separate PR on the other repo?

can do. might be a few days before i get a chance.

Where's the best place to track progress on the new server implementation?

[sestinj]

No server now, everything runs in-process (Node.js) or in the sidebar (browser JS). Most things are moved to the core folder.