Skip to content

feat: local changes #20

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

Merged
1 commit merged into from
Aug 31, 2026
Merged

feat: local changes #20

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
2 changes: 1 addition & 1 deletion apps/dashboard/app/api/infra/action/route.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ export async function POST(request: NextRequest) {
async function triggerAgent(origin: string, service: string, chaosType: string): Promise<void> {
if (process.env.AUTO_TRIGGER_AGENT === 'false') return
try {
await fetch(`${origin}/api/webhook`, {
await fetch(`http://127.0.0.1:3001/api/webhook`, {
method: 'POST',
headers: { 'Content-Type': 'application/json' },
body: JSON.stringify({
Expand Down
4 changes: 4 additions & 0 deletions stop.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,8 @@
# MissionControl — One-Click Stop
Write-Host "Stopping containers..." -ForegroundColor Yellow
docker compose down

Write-Host "Stopping Dashboard..." -ForegroundColor Yellow
Stop-Process -Name "node" -Force -ErrorAction SilentlyContinue

@qodo-code-review qodo-code-review Bot Aug 30, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Action required

1. Kills unrelated node processes 🐞 Bug ☼ Reliability

The new shutdown command force-terminates every Windows process named node, not just
MissionControl's dashboard. Running stop.ps1 can therefore abruptly kill unrelated development
servers, editors, and other Node applications, causing interrupted work or data loss.
Agent Prompt 
## Issue description
`Stop-Process -Name "node" -Force` kills every Windows Node process rather than only the MissionControl dashboard.

## Issue Context
The dashboard runs on port 3001, and the repository already has targeted process-identification logic in `scripts/stop-all.ps1` that verifies the owning process command line before terminating it.

## Fix Focus Areas
- stop.ps1[5-6]
- scripts/stop-all.ps1[8-35]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

@codeant-ai codeant-ai Bot Aug 30, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggestion: Stop-Process -Name "node" forcibly terminates every Node process, including unrelated development servers and applications running on the machine. [logic error]

Assessment: 🟠 Major · 🔁 Occurrence: Sometimes

Use CodeAnt Skill Fix in Cursor Fix in VSCode Claude

Prompt for AI Agent 🤖 
This is a comment left during a code review.

**Path:** stop.ps1
**Line:** 6:6
**Comment:**
	*Logic Error: `Stop-Process -Name "node"` forcibly terminates every Node process, including unrelated development servers and applications running on the machine.

Validate the correctness of the flagged issue. If correct, How can I resolve this? If you propose a fix, implement it and please make it concise.
Once fix is implemented, also check other comments on the same PR, and ask user if the user wants to fix the rest of the comments as well. if said yes, then fetch all the comments validate the correctness and implement a minimal fix
👍 | 👎


Write-Host "Done." -ForegroundColor Green
1 change: 1 addition & 0 deletions update.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
UPDATE agent SET manifest = CAST(json_set(CAST(manifest AS TEXT), '$.model', 'meta/llama-3.3-70b-instruct') AS BLOB) WHERE name = 'missioncontrol';