Skip to content

Implement Azure Local Network Config Tool - Frontend Wizard and Backend CLI #84

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
Copilot wants to merge 19 commits into
base: dev/nl/project_update
Choose a base branch
from
+23,715 −5,644
Draft

Implement Azure Local Network Config Tool - Frontend Wizard and Backend CLI #84

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
f23b087
Initial plan
Copilot Jan 29, 2026
3dffed9
Convert frontend to TypeScript with Vite
Copilot Jan 29, 2026
aa9e6c1
Address code review feedback
Copilot Jan 29, 2026
6d634bc
Complete Phase 2 frontend TypeScript migration with Vite
Copilot Jan 29, 2026
63606af
Implement Phase 3: Python backend CLI for Azure Local network configu…
Copilot Jan 29, 2026
7d29198
Add backend README documentation
Copilot Jan 29, 2026
df06d07
Complete Phase 4 integration testing and documentation
Copilot Jan 29, 2026
6efbd73
update design doc
liunick-msft Jan 29, 2026
b5fbf92
working on UI dev
liunick-msft Jan 30, 2026
e93978c
Pre-Odin UI: Working wizard with Phase 2/3 structure, routing fixes, …
liunick-msft Jan 30, 2026
95d6350
Odin Step 1: Add Configuration Summary Sidebar with real-time updates
liunick-msft Jan 30, 2026
c2f749c
Odin Step 2: Complete dark theme redesign matching Odin UI style
liunick-msft Jan 30, 2026
16f9382
Odin UI: Add dark theme matching Odin website style, fix progress ind…
liunick-msft Jan 30, 2026
a204b9b
Odin UI analysis: Roadmap v9.0 and test refresh (33 tests)
liunick-msft Jan 30, 2026
2ecb13c
Implement Odin UI patterns: numbered sections, breadcrumb nav, theme …
liunick-msft Jan 30, 2026
a6b05ff
UI fixes: remove pattern sidebar, improve link contrast, fix font con…
liunick-msft Jan 30, 2026
186fd03
Single-page scroll layout: unified navigation, improved font controls
liunick-msft Jan 30, 2026
2e2567d
Sticky nav, breadcrumb completion tracking, improved Add buttons, tes…
liunick-msft Jan 30, 2026
51e748f
Fix breadcrumb completion tracking for all sections
liunick-msft Jan 31, 2026
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
51 changes: 51 additions & 0 deletions .github/copilot-instructions.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -212,3 +212,54 @@ project/
- Callouts: `> [!NOTE]`, `> [!TIP]`, `> [!WARNING]`
- Language tags on all code blocks
- `<details>` for optional content

---

## ⚠️ CRITICAL: Environment Safety Rules

> [!WARNING]
> **MANDATORY** rules for all AI agents. Violations will break the development environment.

### Rule 1: NEVER Kill Node/Vite Processes

```bash
# ❌ FORBIDDEN - Will shut down the dev container
pkill -f node
pkill -f vite
pkill -9 node
kill $(pgrep -f vite)
```

**Why:** The dev container depends on Node.js processes. Killing them terminates the container and disconnects your session.

**Safe alternatives:**
- Use Ctrl+C in the terminal running the server
- Close the terminal tab
- Let the process run (it doesn't hurt)

### Rule 2: ALWAYS Use Timeouts

```bash
# ❌ BAD - Can hang forever
npx playwright test
curl http://localhost:3000

# ✅ GOOD - Always wrap with timeout
timeout 120 npx playwright test --reporter=line
timeout 10 curl -s http://localhost:3000
```

**Test timeout requirements:**
- Global: 180s (3 min max total)
- Per-test: 30s
- Per-action: 10s
- Assertions: 5s

```typescript
// In test files
test.setTimeout(30000);
await page.click('#btn', { timeout: 10000 });
await expect(loc).toBeVisible({ timeout: 5000 });
```

**Why:** Hanging processes block CI/CD pipelines and waste development time.
Loading