Skip to content

fix: read Nest controller object paths, skip test routes, describe li… - #146

Merged
itsflaid merged 1 commit into
mainfrom
fix/analyzer-nest-routes-test-scope-library-onboarding
Nov 6, 2026
Merged

itsflaid merged 1 commit into
mainfrom
fix/analyzer-nest-routes-test-scope-library-onboarding

Conversation

@itsflaid

@itsflaid itsflaid commented Nov 6, 2026

Copy link
Copy Markdown
Owner

…braries as libraries

  • nestRouteDetector reads @controller({ path }) and array paths for controllers and methods, so the controller prefix is no longer dropped
  • detectRoutes drops routes declared outside architecture source (specs, e2e, integration/, sample/)
  • onboarding tagline and summary call a package with no app, API or CLI a library instead of a "Personal app"
  • add tests for all three and record the work in the changelog and personal notes

Claude-Session: https://claude.ai/code/session_01AgTNRXkpcmiaaLEb9KjGz1


name: Pull Request Template
about: Template untuk kontribusi ke DevMap
title: ""
labels: ["needs-review"]

Summary perbaikan

Issue terkait

Checklist sebelum submit

  • devmap doctor lolos (tanpa error)
  • Test pnpm test:cli lolos (semua pass)
  • New commands diupdate di docs/commands.md
  • Tidak ada console.log di command files
  • Registry terms sesuai standar (baca docs/guide/05-signal-registry.md)
  • Code style match dengan CONTRIBUTING.md

Catatan tambahan

…braries as libraries

- nestRouteDetector reads @controller({ path }) and array paths for controllers and methods,
  so the controller prefix is no longer dropped
- detectRoutes drops routes declared outside architecture source (specs, e2e, integration/, sample/)
- onboarding tagline and summary call a package with no app, API or CLI a library instead of a
  "Personal app"
- add tests for all three and record the work in the changelog and personal notes

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AgTNRXkpcmiaaLEb9KjGz1
@vercel

vercel Bot commented Nov 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
devmap-web Ready Ready Preview Nov 6, 2026 12:13pm UTC

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

✅ Welcome ke DevMap!

Terima kasih sudah buat PR. Pastikan checklist ini terpenuhi sebelum review:

  • devmap doctor lolos (tanpa error)
  • Test pnpm test:cli lolos (semua pass)
  • New commands diupdate di docs/commands.md
  • Tidak ada console.log di command files
  • Registry terms sesuai standar (baca docs/guide/05-signal-registry.md)
  • Code style match dengan CONTRIBUTING.md

CI akan jalan otomatis pada setiap update. Kalau gagal, akan muncul link run di komentar ini.

Kalau DevMap membantu, tolong star ya: https://github.com/itsflaid/devmap

Terima kasih kontribusinya! 🙏

@itsflaid
itsflaid merged commit 983dd58 into main Nov 6, 2026
3 checks passed
@itsflaid
itsflaid deleted the fix/analyzer-nest-routes-test-scope-library-onboarding branch November 6, 2026 12:13
@itsflaid itsflaid mentioned this pull request Nov 6, 2026
6 tasks

This branch was successfully deployed

1 active deployment
Preview — b941bc3c Deployed Nov 6, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant