Sweep: In the ConnectionInterface.vue, make it clear when a connection problem is from the SSH tunnel

Here's the PR! https://github.com/wwzeng1/beekeeper-studio/pull/3.

💎 Sweep Pro: I'm using GPT-4. You have unlimited GPT-4 tickets. (tracking ID: 103072ea80)

Actions (click)

[ ] ↻ Restart Sweep

Sandbox Execution X

Here are the sandbox execution logs prior to making any changes:

Sandbox logs for eff2485

yarn install 1/1 ❌ (`1`)

yarn install v1.22.19
[1/4] Resolving packages...
warning Lockfile has incorrect entry for "watchpack@^1.7.4". Ignoring it.
[2/4] Fetching packages...
warning url-loader@1.1.2: Invalid bin field for "url-loader".
error @achrinza/node-ipc@9.2.2: The engine "node" is incompatible with this module. Expected version "8 || 10 || 12 || 14 || 16 || 17". Got "18.17.0"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Sandbox failed, so all sandbox checks will be disabled for this issue.

Install Sweep Configs: Pull Request

Step 1: 🔎 Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.

https://github.com/wwzeng1/beekeeper-studio/blob/eff248585cbf93e016a47d8b2d4414734cb67622/apps/studio/src/components/ConnectionInterface.vue#L154-L405 https://github.com/wwzeng1/beekeeper-studio/blob/eff248585cbf93e016a47d8b2d4414734cb67622/apps/studio/docs/guide/README.md#L5-L85 https://github.com/wwzeng1/beekeeper-studio/blob/eff248585cbf93e016a47d8b2d4414734cb67622/apps/studio/src/components/connection/CommonAdvanced.vue#L1-L155 https://github.com/wwzeng1/beekeeper-studio/blob/eff248585cbf93e016a47d8b2d4414734cb67622/apps/studio/src/lib/db/server.ts#L13-L84 https://github.com/wwzeng1/beekeeper-studio/blob/eff248585cbf93e016a47d8b2d4414734cb67622/LICENSE.md#L590-L674

Step 2: ⌨️ Coding

[X] Modify apps/studio/src/components/ConnectionInterface.vue ✓ https://github.com/wwzeng1/beekeeper-studio/commit/f0833664cdca16113502cbd4fa2d754f56516eca
Modify apps/studio/src/components/ConnectionInterface.vue with contents:
• In the ConnectionInterface.vue file, locate the methods 'submit' and 'testConnection' where the connection errors are being handled.
• Modify the catch block in these methods to differentiate between SSH and database errors.
• Import the 'createServer' function from 'apps/studio/src/lib/db/server.ts' file.
• Use the 'createServer' function to create a server instance with the current configuration.
• If the server instance creation fails, it means the error is from the SSH tunnel. In this case, prepend the error message with the string 'SSH Error: '.
• If the server instance creation is successful but the connection fails, it means the error is from the database. In this case, prepend the error message with the string 'Database Error: '.
• Assign the modified error message to 'this.connectionError'.

Step 3: 🔁 Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/ssh-db-error-handling.

🎉 Latest improvements to Sweep:

Sweep now uses the rope library to refactor Python! Check out Large Language Models are Bad at Refactoring Code. To have Sweep refactor your code, try sweep: Refactor <your_file>.py!
Sweep finds and fixes bugs in your code by writing unit tests! Check out Having GPT-4 Iterate on Unit Tests like a Human.

💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. ^{Join Our Discord}

wwzeng1 / beekeeper-studio