Repository navigation
fix(analyzer): audit fixes for v0.9.3 (analyzer, AI timeouts, domain grounding, onboarding) - #150
Merged
Merged
Conversation
- Vue SFC: merge <script> and <script setup> so imports from both blocks reach the graph - Entry points: tooling config files (vitest.config.e2e.ts, app.config.ts) are never entries; Angular's app.config.ts still is - Framework: next.config/nuxt.config inside examples/ no longer decide the framework - Features: images and fonts are not evidence for a feature - Analyzer: skip files with a line over 50k chars (one-line bundles stalled ts-morph for minutes) - AI: bound every chat request with a timeout (DEVMAP_AI_TIMEOUT_MS, default 120s) - onboarding: rank features instead of taking the alphabetical first, dedupe the stack line, fix double period, reject unsupported --language - scanner: ONBOARDING.md written by devmap no longer makes the snapshot stale Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AgTNRa65oer96NSudK8dWf
…ship in the snapshot - domain/summary that share no word with entities, routes or features are ignored and reported in `warnings` - AI features that only touch User/Session/Account-style entities are dropped - `ownership` (from the entity graph) is stored in the snapshot; "Personal app" now needs that evidence instead of being the default Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AgTNRa65oer96NSudK8dWf
…ne name A page is named after its URL segment (Doc) and a role after the concept (Documentation), so the same thing was reported twice. featureStem now aliases doc -> documentation and the merged feature keeps the concept's name. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AgTNRa65oer96NSudK8dWf
…y detected framework - packages: ['.'] (astro-paper) named the root itself and made the project a monorepo. - project.frameworks is the per-category winner that gates route detection, so a Next.js + Expo + TanStack Start monorepo listed only nextjs. The new optional project.detectedFrameworks lists every framework (implied ones such as the React inside Next.js are folded away); doctor, analyze and .devmap/index.json show it. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AgTNRa65oer96NSudK8dWf
…vocabulary features D6: a file TypeScript recovered from (parseDiagnostics) is reported at medium, not high. D7: a registry signal that only a file name or local import supports (no npm package names it) is capped at medium unless a route is served from its files. A scoped package counts through its scope, so @standard-schema/spec is no evidence for Testing. Also syncs CLAUDE.md with the new snapshot fields and engineering notes. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AgTNRa65oer96NSudK8dWf
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Welcome ke DevMap!Terima kasih sudah buat PR. Pastikan checklist ini terpenuhi sebelum review:
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! 🙏 |
Merged
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes from a full test pass of v0.9.3 (19 public repos across Next, Nuxt, SvelteKit, Remix, Astro, Vue, Nest, Hono, Express, t3-turbo, tRPC, excalidraw, Django, Flask, Laravel, Gin, Spring; every CLI command; the MCP server; a mock AI server for failure modes).
Analyzer
<script>and<script setup>read every block, so graph edges and impact are complete.*.config.e2e.ts, tooling configs) are no longer entry points; Angularapp.config.tsstays one when its siblings exist.next.config/nuxt.configdetection uses source files only, soexamples/with-nextjsno longer makes excalidraw a Next.js project.PayPal.svgno longer yields a Payments feature).low): a one-line 1.7 MB bundle madeanalyzehang for over 120 s, now 1.6 s.mediuminstead ofhigh.packages: ["."]no longer makes a monorepo./docspage and the Documentation role are one feature, named "Documentation".mediumunless a route is served from their files. A scoped package counts through its scope, so@standard-schema/specis no evidence for Testing.Project metadata
project.detectedFrameworkslists every framework with signals.project.frameworks(winner per category, gates route detection) is unchanged.doctor,analyzeand.devmap/index.jsonshow the full list; older snapshots fall back toframeworks.AI
DEVMAP_AI_TIMEOUT_MS(default 120 s).explain,flowanddevmap_explainno longer hang;analyzestill completes.ownership); "Personal app" wording needssingle_user_isolated.Onboarding
onboarding --writeno longer makes the snapshot stale; an unsupported--languageis an error.Behavior to know about
mediumrather than being removed. For exampleAuthenticationin vue-realworld is nowmedium: no vendor package and no route is served from its files.vitestnext tospec); a package named by a generic word still counts..devmap/snapshots are regenerated only after a version bump. Useanalyze --freshmeanwhile.Verification
pnpm testinpackages/cli: 588 pass, 0 fail (559 before, 29 added; typecheck included)node packages/cli/test/mcp-smoke.mjs: passes (7 tools)pnpm test:package-e2e: passesframeworksare identical everywhere. Differences are only the intended ones:Docmerged intoDocumentation, astro-papersingle-package,detectedFrameworksadded, and name-only features atmedium(SvelteKit, tRPC, excalidraw, taxonomy, astro-paper, vue-realworld).Not in this PR
No version bump, tag or release.
CLAUDE.mdis synced with the new fields and notes.Generated by Claude Code