Expand Blade sprint with detailed tasks and update code action status

Author: AJenbo

docs/todo.md

@@ -62,14 +62,16 @@ within the same impact tier.
 
 ## Sprint 8 — Blade support
 
-| #   | Item                                             | Impact | Effort    |
-| --- | ------------------------------------------------ | ------ | --------- |
-|     | Clear [refactoring gate](todo/refactor.md)       | —      | —         |
-| BL1 | [Blade template language support](todo/blade.md) | High   | Very High |
-
-The Blade sprint is a placeholder. Scope will be refined after Sprint 7
-ships. The goal is to provide a single-binary PHP + Blade language
-server with Blade intelligence.
+| #   | Item                                                                                                           | Impact | Effort |
+| --- | -------------------------------------------------------------------------------------------------------------- | ------ | ------ |
+|     | Clear [refactoring gate](todo/refactor.md)                                                                     | —      | —      |
+| BL1 | [Blade-aware code actions](todo/blade.md#8-blade-aware-code-actions)                                           | Medium | Medium |
+| BL2 | [Template and component file discovery](todo/blade.md#9-template-and-component-file-discovery)                  | High   | Medium |
+| BL3 | [Component tag parsing (`<x-...>`, `<livewire:...>`, `@props`)](todo/blade.md#10-x-component-tag-parsing-in-preprocessor) | High   | High   |
+| BL4 | [Component and view name completion](todo/blade.md#13-component-and-view-name-completion)                       | High   | Medium |
+| BL5 | [Go-to-definition for view names and components](todo/blade.md#15-go-to-definition-for-view-names-and-components) | Medium | Medium |
+| BL6 | [`@extends` signature merging and component class typing](todo/blade.md#16-signature-merging-for-extends)       | Medium | High   |
+| BL7 | [Blade directive completion](todo/blade.md#19-directive-name-completion)                                        | Medium | Low    |
 
 # Backlog
 

docs/todo/blade.md

@@ -51,18 +51,20 @@ Blade templates into virtual PHP line-by-line, with a source map for
 coordinate translation. The LSP pipeline (`with_file_content`,
 `update_ast`, `did_close`) transparently handles Blade files.
 
-Known issues in the current implementation:
+---
 
-### Code actions are not Blade-aware
+## Phase 2: Component Support
 
-Code actions like "Import class" insert a `use` statement at the top
-of the file rather than inside a `@php` / `<?php` block. All code
-actions that produce text edits need their ranges translated, and
-actions that generate new code need to be aware of Blade structure.
+### 8. Blade-aware code actions
 
----
+Code actions are currently disabled for `.blade.php` files because
+text edits target virtual PHP coordinates and actions like "Import
+class" insert `use` statements at the top of the file rather than
+inside a `@php` / `<?php` block. Re-enable code actions with:
 
-## Phase 2: Component Support
+- Range translation (virtual PHP → Blade) for all text edits.
+- Blade-aware code generation (e.g. insert `use` inside `@php`).
+- Filtering out actions that don't make sense in Blade context.
 
 ### 9. Template and component file discovery
 
@@ -431,19 +433,14 @@ Extend `tests/completion_blade.rs`:
 
 ## Implementation Sequence
 
-Steps 1-2 are complete. The remaining steps build on the existing
-preprocessor and LSP pipeline.
-
-### Step 3: Remaining Phase 1 items
-
-Inject `$loop` inside `@foreach` blocks. Handle `@session`/`@error`/
-`@context` implicit variables. Implement stub directives and verbatim
-regions. Add `languageId` check.
+Phase 1 is complete (steps 1-3): the preprocessor, LSP pipeline
+integration, source mapping, `$loop`/`@session`/`@error`/`@context`
+implicit variables, stub directives, verbatim regions, `languageId`
+check, and code action suppression are all shipped.
 
-**Deliverable:** `$loop->first`, `$value` inside `@session` blocks,
-and `$message` inside `@error` blocks all produce completions.
+The remaining steps build on the existing preprocessor:
 
-### Step 4: Discovery (item 9)
+### Step 4: Discovery (items 8-9)
 
 Implement `src/blade/discovery.rs`. Scan `resources/views/`,
 `app/View/Components/`, `app/Livewire/` at init time. Add the three

src/server.rs

@@ -722,6 +722,13 @@ impl LanguageServer for Backend {
     async fn code_action(&self, params: CodeActionParams) -> Result<Option<CodeActionResponse>> {
         let uri = params.text_document.uri.to_string();
 
+        // Code actions are not yet Blade-aware (edits target virtual PHP
+        // coordinates and may insert code outside valid PHP regions).
+        // Disabled until Phase 2 component support lands.
+        if self.is_blade_file(&uri) {
+            return Ok(None);
+        }
+
         let backend = self.clone_for_blocking();
         let uri_clone = uri.clone();
         tokio::task::spawn_blocking(move || {