fix: remove core/ allowlist, migrate last 2 types, fix cascading

Removed builder.go and types.go from TestCoreStructure allowlist.
Moved sync/core Action to validate/ and watch/core ContextUpdate
to apply/. Created builder/doc.go. Renamed stuttery functions:
DetectBuilder→Detect, FindBuilder→Find, BuilderNames→Names.
Deleted dead entity/claude.go and entity.IndexEntryBlock.

Zero allowlists remain across all 37 tests.

Signed-off-by: Jose Alekhinne <jose@ctx.ist>

Author: josealekhine

internal/audit/core_structure_test.go

@@ -15,17 +15,8 @@ import (
 
 // allowedCoreFiles lists the files permitted directly
 // in a core/ directory (not in subdirectories).
-// builder.go: dep/core houses the interface + registry
-//
-//	that imports language subpackages.
-//
-// types.go: shared types used by multiple subpackages
-//
-//	where moving would create circular imports.
 var allowedCoreFiles = map[string]bool{
-	"doc.go":     true,
-	"builder.go": true,
-	"types.go":   true,
+	"doc.go": true,
 }
 
 // TestCoreStructure ensures core/ directories contain

internal/cli/dep/cmd/root/run.go

@@ -9,9 +9,9 @@ package root
 import (
 	"strings"
 
+	"github.com/ActiveMemory/ctx/internal/cli/dep/core/builder"
 	"github.com/spf13/cobra"
 
-	"github.com/ActiveMemory/ctx/internal/cli/dep/core"
 	"github.com/ActiveMemory/ctx/internal/cli/dep/core/render"
 	"github.com/ActiveMemory/ctx/internal/config/fmt"
 	"github.com/ActiveMemory/ctx/internal/config/token"
@@ -45,22 +45,25 @@ func Run(
 		return errConfig.UnknownFormat(format, supportedFormats)
 	}
 
-	var builder core.GraphBuilder
+	var b builder.GraphBuilder
 	if projType != "" {
-		builder = core.FindBuilder(projType)
-		if builder == nil {
-			names := strings.Join(core.BuilderNames(), token.CommaSpace)
+		b = builder.Find(projType)
+		if b == nil {
+			names := strings.Join(builder.Names(), token.CommaSpace)
 			return errConfig.UnknownProjectType(projType, names)
 		}
 	} else {
-		builder = core.DetectBuilder()
-		if builder == nil {
-			deps.InfoNoProject(cmd, strings.Join(core.BuilderNames(), token.CommaSpace))
+		b = builder.Detect()
+		if b == nil {
+			names := strings.Join(
+				builder.Names(), token.CommaSpace,
+			)
+			deps.InfoNoProject(cmd, names)
 			return nil
 		}
 	}
 
-	graph, buildErr := builder.Build(external)
+	graph, buildErr := b.Build(external)
 	if buildErr != nil {
 		return buildErr
 	}

internal/cli/dep/core/builder.go internal/cli/dep/core/builder/builder.gointernal/cli/dep/core/builder.go renamed to internal/cli/dep/core/builder/builder.go

@@ -4,7 +4,7 @@
 //   \    Copyright 2026-present Context contributors.
 //                 SPDX-License-Identifier: Apache-2.0
 
-package core
+package builder
 
 import (
 	"github.com/ActiveMemory/ctx/internal/cli/dep/core/golang"
@@ -39,12 +39,12 @@ var Builders = []GraphBuilder{
 	&rust.Builder{},
 }
 
-// DetectBuilder returns the first builder whose Detect()
+// Detect returns the first builder whose Detect()
 // returns true, or nil if no ecosystem is detected.
 //
 // Returns:
 //   - GraphBuilder: The detected builder, or nil
-func DetectBuilder() GraphBuilder {
+func Detect() GraphBuilder {
 	for _, b := range Builders {
 		if b.Detect() {
 			return b
@@ -53,15 +53,15 @@ func DetectBuilder() GraphBuilder {
 	return nil
 }
 
-// FindBuilder returns the builder matching the given name,
+// Find returns the builder matching the given name,
 // or nil.
 //
 // Parameters:
 //   - name: Ecosystem name to find (e.g. "go", "node")
 //
 // Returns:
 //   - GraphBuilder: The matching builder, or nil
-func FindBuilder(name string) GraphBuilder {
+func Find(name string) GraphBuilder {
 	for _, b := range Builders {
 		if b.Name() == name {
 			return b
@@ -70,12 +70,12 @@ func FindBuilder(name string) GraphBuilder {
 	return nil
 }
 
-// BuilderNames returns all registered builder names for
+// Names returns all registered builder names for
 // help text.
 //
 // Returns:
 //   - []string: Ordered list of ecosystem names
-func BuilderNames() []string {
+func Names() []string {
 	names := make([]string, len(Builders))
 	for i, b := range Builders {
 		names[i] = b.Name()

internal/cli/dep/core/builder/doc.go

@@ -0,0 +1,13 @@
+//   /    ctx:                         https://ctx.ist
+// ,'`./    do you remember?
+// `.,'\
+//   \    Copyright 2026-present Context contributors.
+//                 SPDX-License-Identifier: Apache-2.0
+
+// Package builder defines the dependency graph builder
+// interface and registry for supported ecosystems.
+//
+// Each ecosystem (Go, Node, Python, Rust) implements
+// the [Builder] interface. [Detect] auto-detects the
+// project type; [Find] looks up a builder by name.
+package builder

internal/cli/dep/core/deps_test.go

@@ -10,9 +10,11 @@ import (
 	"os"
 	"path/filepath"
 	"testing"
+
+	"github.com/ActiveMemory/ctx/internal/cli/dep/core/builder"
 )
 
-func TestDetectBuilder(t *testing.T) {
+func TestDetect(t *testing.T) {
 	orig, getErr := os.Getwd()
 	if getErr != nil {
 		t.Fatal(getErr)
@@ -24,9 +26,9 @@ func TestDetectBuilder(t *testing.T) {
 	if chdirErr := os.Chdir(tmp); chdirErr != nil {
 		t.Fatal(chdirErr)
 	}
-	if b := DetectBuilder(); b != nil {
+	if b := builder.Detect(); b != nil {
 		t.Errorf(
-			"DetectBuilder() = %q in empty dir, want nil",
+			"Detect() = %q in empty dir, want nil",
 			b.Name(),
 		)
 	}
@@ -39,16 +41,16 @@ func TestDetectBuilder(t *testing.T) {
 	if writeErr != nil {
 		t.Fatal(writeErr)
 	}
-	if b := DetectBuilder(); b == nil ||
+	if b := builder.Detect(); b == nil ||
 		b.Name() != "go" {
 		t.Errorf(
-			"DetectBuilder() with go.mod: want 'go', got %v",
+			"Detect() with go.mod: want 'go', got %v",
 			b,
 		)
 	}
 }
 
-func TestDetectBuilder_Node(t *testing.T) {
+func TestDetect_Node(t *testing.T) {
 	orig, getErr := os.Getwd()
 	if getErr != nil {
 		t.Fatal(getErr)
@@ -67,16 +69,16 @@ func TestDetectBuilder_Node(t *testing.T) {
 	if writeErr != nil {
 		t.Fatal(writeErr)
 	}
-	if b := DetectBuilder(); b == nil ||
+	if b := builder.Detect(); b == nil ||
 		b.Name() != "node" {
 		t.Errorf(
-			"DetectBuilder() with package.json: want 'node', got %v",
+			"Detect() with package.json: want 'node', got %v",
 			b,
 		)
 	}
 }
 
-func TestDetectBuilder_Python(t *testing.T) {
+func TestDetect_Python(t *testing.T) {
 	orig, getErr := os.Getwd()
 	if getErr != nil {
 		t.Fatal(getErr)
@@ -95,16 +97,16 @@ func TestDetectBuilder_Python(t *testing.T) {
 	if writeErr != nil {
 		t.Fatal(writeErr)
 	}
-	if b := DetectBuilder(); b == nil ||
+	if b := builder.Detect(); b == nil ||
 		b.Name() != "python" {
 		t.Errorf(
-			"DetectBuilder() with requirements.txt: want 'python', got %v",
+			"Detect() with requirements.txt: want 'python', got %v",
 			b,
 		)
 	}
 }
 
-func TestDetectBuilder_Rust(t *testing.T) {
+func TestDetect_Rust(t *testing.T) {
 	orig, getErr := os.Getwd()
 	if getErr != nil {
 		t.Fatal(getErr)
@@ -124,16 +126,16 @@ func TestDetectBuilder_Rust(t *testing.T) {
 	if writeErr != nil {
 		t.Fatal(writeErr)
 	}
-	if b := DetectBuilder(); b == nil ||
+	if b := builder.Detect(); b == nil ||
 		b.Name() != "rust" {
 		t.Errorf(
-			"DetectBuilder() with Cargo.toml: want 'rust', got %v",
+			"Detect() with Cargo.toml: want 'rust', got %v",
 			b,
 		)
 	}
 }
 
-func TestDetectBuilder_PriorityOrder(t *testing.T) {
+func TestDetect_PriorityOrder(t *testing.T) {
 	orig, getErr := os.Getwd()
 	if getErr != nil {
 		t.Fatal(getErr)
@@ -160,37 +162,37 @@ func TestDetectBuilder_PriorityOrder(t *testing.T) {
 		t.Fatal(writeErr)
 	}
 
-	if b := DetectBuilder(); b == nil ||
+	if b := builder.Detect(); b == nil ||
 		b.Name() != "go" {
 		t.Errorf(
-			"DetectBuilder() with go.mod+package.json: want 'go', got %v",
+			"Detect() with go.mod+package.json: want 'go', got %v",
 			b,
 		)
 	}
 }
 
-func TestFindBuilder(t *testing.T) {
+func TestFind(t *testing.T) {
 	for _, name := range []string{
 		"go", "node", "python", "rust",
 	} {
-		if b := FindBuilder(name); b == nil {
+		if b := builder.Find(name); b == nil {
 			t.Errorf(
-				"FindBuilder(%q) = nil, want builder", name,
+				"Find(%q) = nil, want builder", name,
 			)
 		}
 	}
-	if b := FindBuilder("java"); b != nil {
+	if b := builder.Find("java"); b != nil {
 		t.Errorf(
-			"FindBuilder('java') = %v, want nil", b,
+			"Find('java') = %v, want nil", b,
 		)
 	}
 }
 
-func TestBuilderNames(t *testing.T) {
-	names := BuilderNames()
+func TestNames(t *testing.T) {
+	names := builder.Names()
 	if len(names) != 4 {
 		t.Fatalf(
-			"BuilderNames() returned %d names, want 4",
+			"Names() returned %d names, want 4",
 			len(names),
 		)
 	}
@@ -200,7 +202,7 @@ func TestBuilderNames(t *testing.T) {
 	for i, want := range expected {
 		if names[i] != want {
 			t.Errorf(
-				"BuilderNames()[%d] = %q, want %q",
+				"Names()[%d] = %q, want %q",
 				i, names[i], want,
 			)
 		}

internal/cli/dep/core/doc.go

@@ -7,8 +7,8 @@
 // Package core contains dependency graph building and rendering
 // for Go, Node.js, and Rust ecosystems.
 //
-// [DetectBuilder] auto-selects the right builder based on project
-// files. [FindBuilder] looks up by name. Each ecosystem has a
+// [builder.Detect] auto-selects the right builder based on project
+// files. [builder.Find] looks up by name. Each ecosystem has a
 // GraphBuilder implementation that produces a directed graph.
 // [MermaidID] sanitizes package names for Mermaid diagram syntax.
 package core

internal/cli/sync/core/action/action.go

@@ -7,7 +7,6 @@
 package action
 
 import (
-	"github.com/ActiveMemory/ctx/internal/cli/sync/core"
 	"github.com/ActiveMemory/ctx/internal/cli/sync/core/validate"
 	"github.com/ActiveMemory/ctx/internal/entity"
 )
@@ -25,8 +24,8 @@ import (
 //
 // Returns:
 //   - []Action: List of suggested actions to reconcile context with codebase
-func Detect(ctx *entity.Context) []core.Action {
-	var actions []core.Action
+func Detect(ctx *entity.Context) []validate.Action {
+	var actions []validate.Action
 
 	// Check for new top-level directories not mentioned in ARCHITECTURE.md
 	actions = append(actions, validate.CheckNewDirectories(ctx)...)

internal/cli/sync/core/core_test.go

@@ -13,7 +13,6 @@ import (
 	"testing"
 
 	"github.com/ActiveMemory/ctx/internal/cli/initialize"
-	"github.com/ActiveMemory/ctx/internal/cli/sync/core"
 	"github.com/ActiveMemory/ctx/internal/cli/sync/core/action"
 	"github.com/ActiveMemory/ctx/internal/cli/sync/core/validate"
 	"github.com/ActiveMemory/ctx/internal/config/ctx"
@@ -320,7 +319,7 @@ func TestCheckPackageFiles_ArchContainsDependencies(t *testing.T) {
 }
 
 func TestAction_Fields(t *testing.T) {
-	a := core.Action{
+	a := validate.Action{
 		Type:        "NEW_DIR",
 		File:        ctx.Architecture,
 		Description: "test description",

internal/cli/sync/core/types.go internal/cli/sync/core/validate/types.gointernal/cli/sync/core/types.go renamed to internal/cli/sync/core/validate/types.go

@@ -4,12 +4,13 @@
 //   \    Copyright 2026-present Context contributors.
 //                 SPDX-License-Identifier: Apache-2.0
 
-package core
+package validate
 
 // Action represents a suggested sync action.
 //
 // Fields:
-//   - Type: Category of action (e.g., "DEPS", "CONFIG", "NEW_DIR")
+//   - Type: Category of action
+//     (e.g., "DEPS", "CONFIG", "NEW_DIR")
 //   - File: Context file that should be updated
 //   - Description: What was detected that needs attention
 //   - Suggestion: Recommended action to take