Skip to content

fix: include HTML/CSS files in keyword-scored file selection - #40

Merged
suyog19 merged 8 commits into
mainfrom
dev
Apr 25, 2026
Merged

fix: include HTML/CSS files in keyword-scored file selection#40
suyog19 merged 8 commits into
mainfrom
dev

Conversation

@suyog19

@suyog19 suyog19 commented Apr 25, 2026

Copy link
Copy Markdown
Owner

Summary

EXTENSION_TO_LANGUAGE in repo_analysis.py is imported by claude_client.py and used as the gate for which files enter the keyword scorer in _select_files_for_story. Without .html/.css in the map, static site files like frontend/dist/index.html are silently skipped — Claude is never given them, and defaults to editing README.

Root cause found via SJ-3: story required updating landing page caption in frontend/dist/index.html (plain HTML static site, no package.json). File scanner reported {'.py':19, '.md':17, '.txt':7} — no HTML. Keyword scorer had no HTML candidates; matched README instead. All three agents correctly blocked the resulting PR.

Files changed

  • app/repo_analysis.py — added .html, .htm, .css, .vue, .svelte

🤖 Generated with Claude Code

suyog19 and others added 8 commits April 25, 2026 19:58
Adds POST /admin/ui/projects/{repo_slug}/rescan route that creates a
new onboarding run and enqueues it, then redirects to the live status
page — same flow as the wizard but without touching the Jira mapping.
Adds CSRF-protected button in the project detail page header.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ity check, rescan

- repo_profiler: document _scan_for_subdir_stacks() and monorepo detection
- claude_client: document file_landmark_map in arch tool, detect_epic_missing_specifics (Haiku), plan_epic_breakdown full-context params
- Capability Profiles: add Monorepo detection note
- Clarification Loop: add Missing specifics trigger (Epic) row
- Project knowledge injection: document file_landmarks bullet
- Dashboard: document Re-run Analysis button

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Users naturally append reason text (e.g. "REGENERATE 1 caption was wrong").
Align _APPROVAL_RE with _CLARIFICATION_RE which already supports this.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
When user sends "REGENERATE 1 <reason>", the trailing text is now stored
as an epic-scoped manual_note so Claude incorporates the feedback on the
regenerated breakdown instead of ignoring it.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Previously the approval message only showed story titles, so users had
no way to verify the specifics without checking the DB. Now each story
block includes its description (truncated at 300 chars) so the from/to
values and key details are visible before approving.

Also clarifies REGENERATE supports optional feedback text.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
EXTENSION_TO_LANGUAGE is used as the gate in _select_files_for_story.
Without .html, static site files like frontend/dist/index.html are
silently skipped and never reach the keyword scorer — so Claude is
never given the right file to edit for frontend/content changes.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@suyog19
suyog19 merged commit aecc6f5 into main Apr 25, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant