devrites 6.0.4 → 6.1.0

package/engine/internal/lib/cursor_compat_test.go

@@ -1,65 +0,0 @@
-package lib
-
-import (
-	"os"
-	"path/filepath"
-	"strings"
-	"testing"
-
-	"github.com/devrites/devrites/internal/state"
-)
-
-const canonicalCursor = `# State
-
-## Cursor
-| Key | Value |
-| --- | --- |
-| phase | temper |
-| status | running |
-| next_action | /rite-define |
-| plan_approved | 2026-07-20 |
-| afk_slices_remaining | 2 |
-`
-
-func TestOfficialBulletCursorAndKeyAliasesRemainReadable(t *testing.T) {
-	lines := strings.Split("- Phase: build\n- Next step: continue\n- qid: Q-001\n", "\n")
-	for key, want := range map[string]string{
-		state.CursorPhase:      "build",
-		state.CursorNextAction: "continue",
-		state.CursorQuestionID: "Q-001",
-	} {
-		if got, ok := state.CursorField(lines, key); !ok || got != want {
-			t.Fatalf("CursorField(%q)=(%q,%v), want %q", key, got, ok, want)
-		}
-	}
-}
-
-func TestClearAwaitingSupportsCanonicalCursor(t *testing.T) {
-	path := filepath.Join(t.TempDir(), "state.md")
-	state := canonicalCursor + `
-## Awaiting human
-| Key | Value |
-| --- | --- |
-| question_id | Q-001 |
-| gate | blocking |
-`
-	state = strings.Replace(state, "| status | running |", "| status | awaiting_human |", 1)
-	if err := os.WriteFile(path, []byte(state), 0o644); err != nil {
-		t.Fatal(err)
-	}
-
-	if err := clearAwaiting(path, "Q-001"); err != nil {
-		t.Fatal(err)
-	}
-	raw, err := os.ReadFile(path)
-	if err != nil {
-		t.Fatal(err)
-	}
-	text := string(raw)
-	if strings.Contains(text, "## Awaiting human") || !strings.Contains(text, "| status | running |") {
-		t.Fatalf("canonical awaiting state was not cleared:\n%s", text)
-	}
-	if !strings.Contains(text, "$rite-temper") || strings.Contains(text, "$rite-build") {
-		t.Fatalf("canonical awaiting state resumed the wrong phase:\n%s", text)
-	}
-}

package/engine/internal/lib/detect_test.go

@@ -1,118 +0,0 @@
-package lib
-
-import (
-	"bytes"
-	"encoding/json"
-	"strings"
-	"testing"
-)
-
-func runDetect(t *testing.T, root string, args ...string) (int, string) {
-	t.Helper()
-	stdout, stderr := &bytes.Buffer{}, &bytes.Buffer{}
-	full := append([]string{"commands", "--root", root}, args...)
-	code := RunDetectCommands("", full, stdout, stderr)
-	return code, stdout.String() + stderr.String()
-}
-
-func TestDetectMakefileWins(t *testing.T) {
-	root := t.TempDir()
-	writeFile(t, root+"/Makefile", "test:\n\tgo test ./internal/...\n\nlint:\n\tgolangci-lint run\n\nbuild:\n\tgo build ./cmd/x\n")
-	writeFile(t, root+"/go.mod", "module example.com/x\n\ngo 1.25\n")
-	code, out := runDetect(t, root)
-	if code != 0 {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-	for _, want := range []string{"make test", "make lint", "make build", "go vet ./...", "go test -tags=integration"} {
-		if !strings.Contains(out, want) {
-			t.Fatalf("missing %q in:\n%s", want, out)
-		}
-	}
-}
-
-func TestDetectPackageJSONAndManager(t *testing.T) {
-	root := t.TempDir()
-	writeFile(t, root+"/package.json", `{"scripts":{"test":"vitest","lint":"eslint .","typecheck":"tsc -b","build":"vite build"}}`)
-	writeFile(t, root+"/pnpm-lock.yaml", "lockfileVersion: '9.0'\n")
-	code, out := runDetect(t, root)
-	if code != 0 {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-	for _, want := range []string{"pnpm test", "pnpm run lint", "pnpm run typecheck", "pnpm run build", "package-manager: pnpm"} {
-		if !strings.Contains(out, want) {
-			t.Fatalf("missing %q in:\n%s", want, out)
-		}
-	}
-	if !strings.Contains(out, "integration") || !strings.Contains(out, "unresolved") {
-		t.Fatalf("integration slot should be unresolved:\n%s", out)
-	}
-}
-
-func TestDetectManifestFallbackAndJSON(t *testing.T) {
-	root := t.TempDir()
-	writeFile(t, root+"/go.mod", "module example.com/y\n\ngo 1.25\n")
-	code, out := runDetect(t, root, "--json")
-	if code != 0 {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-	var report commandReport
-	if err := json.Unmarshal([]byte(out), &report); err != nil {
-		t.Fatalf("bad json %v:\n%s", err, out)
-	}
-	found := map[string]string{}
-	for _, c := range report.Commands {
-		found[c.Name] = c.Command
-	}
-	if found["unit"] != "go test ./..." || found["vet"] != "go vet ./..." {
-		t.Fatalf("unexpected commands: %v", found)
-	}
-}
-
-func TestDetectManifestFallbackSkipsManifestWithoutSlot(t *testing.T) {
-	root := t.TempDir()
-	writeFile(t, root+"/setup.py", "from setuptools import setup\nsetup()\n")
-	writeFile(t, root+"/build.gradle", "plugins { id 'java' }\n")
-	code, out := runDetect(t, root)
-	if code != 0 {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-	if !strings.Contains(out, "gradle build") || !strings.Contains(out, "build.gradle") {
-		t.Fatalf("build slot should resolve from build.gradle:\n%s", out)
-	}
-}
-
-func TestDetectEmptyRepoReportsUnresolved(t *testing.T) {
-	root := t.TempDir()
-	code, out := runDetect(t, root)
-	if code != 0 {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-	if strings.Count(out, "unresolved") != len(commandSlots) {
-		t.Fatalf("expected all slots unresolved:\n%s", out)
-	}
-}
-
-func TestDetectMixedSources(t *testing.T) {
-	root := t.TempDir()
-	writeFile(t, root+"/Makefile", "test:\n\tpytest -q\n")
-	writeFile(t, root+"/package.json", `{"scripts":{"lint":"eslint ."}}`)
-	writeFile(t, root+"/package-lock.json", "{}")
-	writeFile(t, root+"/pyproject.toml", "[project]\nname='x'\n")
-	code, out := runDetect(t, root)
-	if code != 0 {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-	if !strings.Contains(out, "make test") || !strings.Contains(out, "npm run lint") || !strings.Contains(out, "mypy .") {
-		t.Fatalf("mixed-source resolution wrong:\n%s", out)
-	}
-}
-
-func TestDetectUsageError(t *testing.T) {
-	stdout, stderr := &bytes.Buffer{}, &bytes.Buffer{}
-	if code := RunDetectCommands("", nil, stdout, stderr); code != 2 {
-		t.Fatalf("missing subcommand should be usage error, got %d", code)
-	}
-	if code := RunDetectCommands("", []string{"commands", "--bogus"}, stdout, stderr); code != 2 {
-		t.Fatalf("bad flag should be usage error, got %d", code)
-	}
-}

package/engine/internal/lib/drift_test.go

@@ -1,135 +0,0 @@
-package lib
-
-import (
-	"bytes"
-	"os"
-	"path/filepath"
-	"strings"
-	"testing"
-	"time"
-
-	"github.com/devrites/devrites/internal/gate"
-)
-
-// driftWorkspace builds a workspace with every required readiness input.
-func driftWorkspace(t *testing.T) (root, featureDir string) {
-	t.Helper()
-	root, featureDir = newWorkspace(t, "vet")
-	for _, name := range []string{
-		"spec.md", "decision-coverage.md", "architecture.md",
-		"plan.md", "tasks.md", "traceability.md", "test-plan.md",
-	} {
-		writeFile(t, filepath.Join(featureDir, name), "# "+name+"\n\ncontent\n")
-	}
-	return root, featureDir
-}
-
-func checkDrift(t *testing.T, root string, args ...string) (int, string) {
-	t.Helper()
-	stdout, stderr := &bytes.Buffer{}, &bytes.Buffer{}
-	code := RunCheckDrift(root, args, stdout, stderr)
-	return code, stdout.String() + stderr.String()
-}
-
-func TestDriftUnprovenWithoutBaselineOrBinding(t *testing.T) {
-	root, _ := driftWorkspace(t)
-	code, out := checkDrift(t, root, "feat")
-	if code != 0 || !strings.Contains(out, "unproven") {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-}
-
-func TestDriftRecordThenCleanPass(t *testing.T) {
-	root, _ := driftWorkspace(t)
-	code, out := checkDrift(t, root, "feat", "--record")
-	if code != 0 || !strings.Contains(out, "baseline recorded") {
-		t.Fatalf("record code=%d out=%s", code, out)
-	}
-	code, out = checkDrift(t, root, "feat")
-	if code != 0 || !strings.Contains(out, "drift: none") {
-		t.Fatalf("compare code=%d out=%s", code, out)
-	}
-}
-
-func TestDriftAttributesChangedFile(t *testing.T) {
-	root, featureDir := driftWorkspace(t)
-	if code, out := checkDrift(t, root, "feat", "--record"); code != 0 {
-		t.Fatalf("record code=%d out=%s", code, out)
-	}
-	writeFile(t, filepath.Join(featureDir, "plan.md"), "# plan.md\n\nrewritten\n")
-	writeFile(t, filepath.Join(featureDir, "strategy.md"), "# strategy\n\nnew optional input\n")
-	code, out := checkDrift(t, root, "feat")
-	if code != 0 {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-	if !strings.Contains(out, "plan.md changed") || !strings.Contains(out, "strategy.md added") {
-		t.Fatalf("expected per-file attribution:\n%s", out)
-	}
-	if strings.Contains(out, "spec.md changed") {
-		t.Fatalf("spec.md did not change:\n%s", out)
-	}
-}
-
-func TestDriftFallbackEngReviewBinding(t *testing.T) {
-	root, featureDir := driftWorkspace(t)
-	_, binding, err := gate.ReadinessInputDigests(root, "feat")
-	if err != nil {
-		t.Fatal(err)
-	}
-	writeFile(t, filepath.Join(featureDir, "eng-review.md"), "# Eng review\n\n"+binding+"\n")
-	code, out := checkDrift(t, root, "feat")
-	if code != 0 || !strings.Contains(out, "drift: none") {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-	writeFile(t, filepath.Join(featureDir, "tasks.md"), "# tasks\n\ndrifted\n")
-	// eng-review must predate the drifted file for the mtime heuristic.
-	old := time.Now().Add(-time.Hour)
-	if err := os.Chtimes(filepath.Join(featureDir, "eng-review.md"), old, old); err != nil {
-		t.Fatal(err)
-	}
-	code, out = checkDrift(t, root, "feat")
-	if code != 0 {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-	if !strings.Contains(out, "DRIFT: readiness inputs changed") || !strings.Contains(out, "tasks.md") {
-		t.Fatalf("expected aggregate drift + candidate:\n%s", out)
-	}
-}
-
-func TestDriftCorruptBaselineBlocks(t *testing.T) {
-	root, featureDir := driftWorkspace(t)
-	writeFile(t, filepath.Join(featureDir, driftBaselineFile), "{not json")
-	code, out := checkDrift(t, root, "feat")
-	if code != 3 || !strings.Contains(out, "BLOCKED") {
-		t.Fatalf("code=%d out=%s", code, out)
-	}
-}
-
-func TestDriftClassifiesReviewedIdentityOnlyChange(t *testing.T) {
-	root, featureDir := driftWorkspace(t)
-	if code, out := checkDrift(t, root, "feat", "--record"); code != 0 {
-		t.Fatalf("record code=%d out=%s", code, out)
-	}
-	plan := filepath.Join(featureDir, "test-plan.md")
-	line := "Workflow Artifact reviewed identity: " + strings.Repeat("ab", 32) + "\n"
-	writeFile(t, plan, "# test-plan.md\n\ncontent\n"+line)
-	code, out := checkDrift(t, root, "feat")
-	if code != 0 || !strings.Contains(out, "test-plan.md reviewed-identity-only") || !strings.Contains(out, "bytes-only recheck") {
-		t.Fatalf("identity-only change not classified: code=%d\n%s", code, out)
-	}
-	// Any other edit, including a malformed identity line, is ordinary drift.
-	for _, body := range []string{
-		"# test-plan.md\n\nchanged\n" + line,
-		"# test-plan.md\n\ncontent\nWorkflow Artifact reviewed identity: not-a-digest\n",
-	} {
-		writeFile(t, plan, body)
-		if _, out := checkDrift(t, root, "feat"); !strings.Contains(out, "test-plan.md changed") || strings.Contains(out, "bytes-only") {
-			t.Fatalf("non-identity edit hidden:\n%s", out)
-		}
-	}
-	writeFile(t, plan, "# test-plan.md\n\ncontent\n"+line)
-	writeFile(t, filepath.Join(featureDir, "plan.md"), "# plan.md\n\nrewritten\n")
-	if _, out := checkDrift(t, root, "feat"); strings.Contains(out, "bytes-only") {
-		t.Fatalf("a second changed input must void the bytes-only route:\n%s", out)
-	}
-}