Skip to content

Cloude #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
amelianoir wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Cloude #9

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
d6857ad
Create launch.json
amelianoir Nov 15, 2025
cb2ca4b
Create npm-gulp.yml
amelianoir Nov 15, 2025
3450e8c
Create settings.json
amelianoir Nov 15, 2025
fa68b1d
Merge branch 'main' of https://github.com/amelianoir/api-mcp-server
amelianoir Nov 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 28 additions & 0 deletions .github/workflows/npm-gulp.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
name: NodeJS with Gulp

on:
push:
branches: [ "main" ]
pull_request:
branches: [ "main" ]

jobs:
build:
runs-on: ubuntu-latest

strategy:
matrix:
node-version: [18.x, 20.x, 22.x]

steps:
- uses: actions/checkout@v4

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}

- name: Build
run: |
npm install
gulp
35 changes: 35 additions & 0 deletions .vscode/launch.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [


{
"args": [
"--extensionDevelopmentPath=${workspaceFolder}"
],
"name": "Launch Extension",
"outFiles": [
"${workspaceFolder}/out/**/*.js"
],
"preLaunchTask": "npm",
"request": "launch",
"type": "extensionHost"
},
{
"command": "npm start",
"name": "Run npm start",
"request": "launch",
"type": "node-terminal"
},
{
"type": "node-terminal",
"name": "Run Script: start",
"request": "launch",
"command": "npm run start",
"cwd": "${workspaceFolder}"
}
Comment on lines +21 to +33
Copy link

@coderabbitai coderabbitai bot Nov 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Consolidate redundant launch configurations.

Configurations "Run npm start" (lines 21–26) and "Run Script: start" (lines 27–33) are functionally equivalent and create unnecessary maintenance overhead. Both execute essentially the same command (npm start and npm run start), with only minor differences in how they're specified.

Consider keeping only one and removing the other, or clarify the distinct purpose of each if they serve different workflows.

🤖 Prompt for AI Agents 
In .vscode/launch.json around lines 21 to 33, there are two redundant launch
configurations ("Run npm start" and "Run Script: start") that both launch the
app via npm; remove one of them (prefer keeping the one with the most complete
fields, e.g., the entry that includes cwd and descriptive name) or merge
intended differences by consolidating into a single configuration with the
desired name, command ("npm start" or "npm run start"), type "node-terminal",
request "launch" and cwd "${workspaceFolder}", and delete the other duplicate
entry to avoid maintenance overhead.

]
}
6 changes: 6 additions & 0 deletions .vscode/settings.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
{
"[ruby]": {
"diffEditor.codeLens": true,
"editor.formatOnType": true
}
}