Skip to content

Fix HTML class reference extraction - #2599

Merged
Widthdom merged 4 commits into
mainfrom
codex/fix-issue2103
May 24, 2026
Merged

Fix HTML class reference extraction#2599
Widthdom merged 4 commits into
mainfrom
codex/fix-issue2103

Conversation

@Widthdom

@Widthdom Widthdom commented May 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • Split HTML class and className attribute values into individual reference symbols.
  • Added coverage for double spaces, Tailwind-like variants, and empty class attributes.
  • Merged latest main and resolved the HTML markup extractor conflict.
  • Fixed the newly surfaced Rust trait associated type default CI failure; see Rust associated type defaults are not emitted for trait symbols #2603.

Fixes #2103
Refs #2603

Documentation and changelog

  • Added changelog fragment: changelog.d/unreleased/2103.fixed.md
  • Added changelog fragment: changelog.d/unreleased/2603.fixed.md
  • No README/user guide update was needed because these are indexing correctness fixes, not new commands or options.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~SymbolExtractorTests.Extract_HtmlClassAttributes_IndexesIndividualClassReferences
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~SymbolExtractorTests.Extract_RustTraitAssociatedTypeDefaults_RecordsPropertySymbols
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release -f net9.0 --filter FullyQualifiedName~SymbolExtractorTests.Extract_RustTraitAssociatedTypeDefaults_RecordsPropertySymbols -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • codex exec review --base origin/main (initial pass completed before this update; rerun after merge was stopped due node_repl timeout)

# Conflicts:
#	src/CodeIndex/Indexer/Symbols/SymbolExtractor.Markup.cs
@Widthdom
Widthdom merged commit c5fbc16 into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue2103 branch May 24, 2026 10:00
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.

HTML class attribute multi-value extraction collapses tokens

1 participant