fix: migrate 42 magic strings from PR #55 to YAML assets

All user-facing format strings in write/steering, write/skill,
write/trigger, write/setup, mcp/handler, trigger/, and drift/
now go through desc.Text() with YAML-backed DescKeys.

Added 42 YAML entries in ui.yaml and 30 DescKey constants across
6 config/embed/text files (steering.go, trigger.go new; skill.go,
setup.go, mcp_tool.go, drift.go updated). Also adds predicate
naming convention section to audit-conventions.md.

Spec: specs/ast-audit-contributor-guide.md
Signed-off-by: Jose Alekhinne <jose@ctx.ist>

Author: josealekhine

docs/reference/audit-conventions.md

@@ -702,6 +702,41 @@ func Journal(cmd *cobra.Command, ...) {
 
 ---
 
+## Predicate Naming (no `Is`/`Has`/`Can` prefix)
+
+**Test:** None (manual review convention)
+
+Exported methods that return `bool` must not use `Is`, `Has`, or
+`Can` prefixes. The predicate reads more naturally without them,
+especially at call sites where the package name provides context.
+
+**Before:**
+
+```go
+func IsCompleted(t *Task) bool { ... }
+func HasChildren(n *Node) bool { ... }
+func IsExemptPackage(path string) bool { ... }
+```
+
+**After:**
+
+```go
+func Completed(t *Task) bool { ... }
+func Children(n *Node) bool { ... }  // or: ChildCount > 0
+func ExemptPackage(path string) bool { ... }
+```
+
+**Rule:** Drop the prefix. Private helpers may use prefixes when it
+reads more naturally (`isValid` in a local context is fine). This
+convention applies to exported methods and package-level functions.
+See CONVENTIONS.md "Predicates" section.
+
+This is not yet enforced by an AST test — it requires semantic
+understanding of return types and naming intent that makes automated
+detection fragile. Apply during code review.
+
+---
+
 ## Mixed Visibility
 
 **Test:** `TestNoMixedVisibility`

internal/assets/commands/text/ui.yaml

@@ -526,3 +526,94 @@ write.why-label-invariants:
   short: "Design Invariants"
 help.community-footer:
   short: "\nHave a question, bug, or feature request? Join us on Discord:\n  https://ctx.ist/discord\n"
+
+write.steering-created:
+  short: 'Created %s'
+write.steering-skipped:
+  short: 'Skipped %s (already exists)'
+write.steering-init-summary:
+  short: "\n%d created, %d skipped"
+write.steering-file-entry:
+  short: '%-20s  inclusion=%-7s  priority=%-3d  tools=%s'
+write.steering-file-count:
+  short: "\n%d steering file(s)"
+write.steering-preview-head:
+  short: 'Steering files matching prompt %q:'
+write.steering-preview-entry:
+  short: '  %-20s  inclusion=%-7s  priority=%-3d  tools=%s'
+write.steering-preview-count:
+  short: "\n%d file(s) would be included"
+write.steering-sync-written:
+  short: 'Written: %s'
+write.steering-sync-skipped:
+  short: 'Skipped: %s'
+write.steering-sync-error:
+  short: 'Error: %s'
+write.steering-sync-summary:
+  short: "\n%d written, %d skipped, %d errors"
+
+write.skill-installed:
+  short: 'Installed %s → %s'
+write.skill-entry-desc:
+  short: '  %-20s  %s'
+write.skill-entry:
+  short: '  %s'
+write.skill-count:
+  short: "\n%d skill(s)"
+write.skill-removed:
+  short: 'Removed %s'
+
+write.setup-deploy-complete:
+  short: '%s setup complete.'
+write.setup-deploy-mcp:
+  short: '  MCP server: %s'
+write.setup-deploy-steering:
+  short: '  Steering:   %s'
+write.setup-deploy-exists:
+  short: "\u2713 %s already exists (skipped)"
+write.setup-deploy-created:
+  short: "\u2713 Created %s"
+write.setup-deploy-synced:
+  short: "\u2713 Synced steering: %s"
+write.setup-deploy-skip-steer:
+  short: '  Skipped steering: %s (unchanged)'
+
+mcp.steering-section:
+  short: "## %s\n\n%s\n\n"
+mcp.search-hit-line:
+  short: "%s:%d: %s\n"
+mcp.search-no-match:
+  short: 'No matches for %q in %s.'
+
+trigger.warn:
+  short: 'hook %s: %v'
+trigger.error-item:
+  short: '%s: %s'
+trigger.skip-warn:
+  short: 'hook skip %s: %v'
+
+drift.tool-suffix:
+  short: '%s (tool: %s)'
+
+write.trigger-created:
+  short: 'Created %s'
+write.trigger-disabled:
+  short: 'Disabled %s (%s)'
+write.trigger-enabled:
+  short: 'Enabled %s (%s)'
+write.trigger-type-hdr:
+  short: '[%s]'
+write.trigger-entry:
+  short: '  %-20s  %-8s  %s'
+write.trigger-count:
+  short: '%d hook(s)'
+write.trigger-test-hdr:
+  short: 'Testing %s hooks...'
+write.trigger-test-input:
+  short: "Input:\n%s"
+write.trigger-cancelled:
+  short: 'Cancelled: %s'
+write.trigger-context:
+  short: "Context:\n%s"
+write.trigger-err-line:
+  short: '  %s'

internal/config/embed/text/drift.go

@@ -58,6 +58,7 @@ const (
 	DescKeyDriftInvalidTool          = "drift.invalid-tool"
 	DescKeyDriftHookNoExec           = "drift.hook-no-exec"
 	DescKeyDriftStaleSyncFile        = "drift.stale-sync-file"
+	DescKeyDriftToolSuffix           = "drift.tool-suffix"
 	DescKeyVersionDriftRelayMessage  = "version-drift.relay-message"
 	DescKeyWriteVersionDriftFallback = "write.version-drift-fallback"
 )

internal/config/embed/text/mcp_tool.go

@@ -45,3 +45,10 @@ const (
 	DescKeyMCPToolPropSearchQuery   = "mcp.tool-prop-search-query"
 	DescKeyMCPToolPropSummary       = "mcp.tool-prop-summary"
 )
+
+// DescKeys for MCP handler steering/search output.
+const (
+	DescKeyMCPSteeringSection = "mcp.steering-section"
+	DescKeyMCPSearchHitLine   = "mcp.search-hit-line"
+	DescKeyMCPSearchNoMatch   = "mcp.search-no-match"
+)

internal/config/embed/text/setup.go

@@ -8,6 +8,13 @@ package text
 
 // DescKeys for setup wizard write output.
 const (
-	DescKeyWriteSetupDone   = "write.setup-done"
-	DescKeyWriteSetupPrompt = "write.setup-prompt"
+	DescKeyWriteSetupDone            = "write.setup-done"
+	DescKeyWriteSetupPrompt          = "write.setup-prompt"
+	DescKeyWriteSetupDeployComplete  = "write.setup-deploy-complete"
+	DescKeyWriteSetupDeployMCP       = "write.setup-deploy-mcp"
+	DescKeyWriteSetupDeploySteering  = "write.setup-deploy-steering"
+	DescKeyWriteSetupDeployExists    = "write.setup-deploy-exists"
+	DescKeyWriteSetupDeployCreated   = "write.setup-deploy-created"
+	DescKeyWriteSetupDeploySynced    = "write.setup-deploy-synced"
+	DescKeyWriteSetupDeploySkipSteer = "write.setup-deploy-skip-steer"
 )

internal/config/embed/text/skill.go

@@ -8,6 +8,11 @@ package text
 
 // DescKeys for skill display write output.
 const (
-	DescKeyWriteSkillLine    = "write.skill-line"
-	DescKeyWriteSkillsHeader = "write.skills-header"
+	DescKeyWriteSkillLine      = "write.skill-line"
+	DescKeyWriteSkillsHeader   = "write.skills-header"
+	DescKeyWriteSkillInstalled = "write.skill-installed"
+	DescKeyWriteSkillEntryDesc = "write.skill-entry-desc"
+	DescKeyWriteSkillEntry     = "write.skill-entry"
+	DescKeyWriteSkillCount     = "write.skill-count"
+	DescKeyWriteSkillRemoved   = "write.skill-removed"
 )

internal/config/embed/text/steering.go

@@ -0,0 +1,23 @@
+//   /    ctx:                         https://ctx.ist
+// ,'`./    do you remember?
+// `.,'\\
+//   \    Copyright 2026-present Context contributors.
+//                 SPDX-License-Identifier: Apache-2.0
+
+package text
+
+// DescKeys for steering write output.
+const (
+	DescKeyWriteSteeringCreated      = "write.steering-created"
+	DescKeyWriteSteeringSkipped      = "write.steering-skipped"
+	DescKeyWriteSteeringInitSummary  = "write.steering-init-summary"
+	DescKeyWriteSteeringFileEntry    = "write.steering-file-entry"
+	DescKeyWriteSteeringFileCount    = "write.steering-file-count"
+	DescKeyWriteSteeringPreviewHead  = "write.steering-preview-head"
+	DescKeyWriteSteeringPreviewEntry = "write.steering-preview-entry"
+	DescKeyWriteSteeringPreviewCount = "write.steering-preview-count"
+	DescKeyWriteSteeringSyncWritten  = "write.steering-sync-written"
+	DescKeyWriteSteeringSyncSkipped  = "write.steering-sync-skipped"
+	DescKeyWriteSteeringSyncError    = "write.steering-sync-error"
+	DescKeyWriteSteeringSyncSummary  = "write.steering-sync-summary"
+)

internal/config/embed/text/trigger.go

@@ -0,0 +1,29 @@
+//   /    ctx:                         https://ctx.ist
+// ,'`./    do you remember?
+// `.,'\\
+//   \    Copyright 2026-present Context contributors.
+//                 SPDX-License-Identifier: Apache-2.0
+
+package text
+
+// DescKeys for trigger (hook runner) output.
+const (
+	DescKeyTriggerWarn      = "trigger.warn"
+	DescKeyTriggerErrorItem = "trigger.error-item"
+	DescKeyTriggerSkipWarn  = "trigger.skip-warn"
+)
+
+// DescKeys for write/trigger display output.
+const (
+	DescKeyWriteTriggerCreated   = "write.trigger-created"
+	DescKeyWriteTriggerDisabled  = "write.trigger-disabled"
+	DescKeyWriteTriggerEnabled   = "write.trigger-enabled"
+	DescKeyWriteTriggerTypeHdr   = "write.trigger-type-hdr"
+	DescKeyWriteTriggerEntry     = "write.trigger-entry"
+	DescKeyWriteTriggerCount     = "write.trigger-count"
+	DescKeyWriteTriggerTestHdr   = "write.trigger-test-hdr"
+	DescKeyWriteTriggerTestInput = "write.trigger-test-input"
+	DescKeyWriteTriggerCancelled = "write.trigger-cancelled"
+	DescKeyWriteTriggerContext   = "write.trigger-context"
+	DescKeyWriteTriggerErrLine   = "write.trigger-err-line"
+)

internal/drift/check_ext.go

@@ -143,7 +143,9 @@ func checkSyncStaleness(report *Report) {
 				File:    name,
 				Type:    IssueStaleSyncFile,
 				Message: desc.Text(text.DescKeyDriftStaleSyncFile),
-				Path:    fmt.Sprintf("%s (tool: %s)", name, tool),
+				Path: fmt.Sprintf(
+					desc.Text(text.DescKeyDriftToolSuffix),
+					name, tool),
 			})
 			found = true
 		}

internal/mcp/handler/steering.go

@@ -14,6 +14,8 @@ import (
 	"path/filepath"
 	"strings"
 
+	"github.com/ActiveMemory/ctx/internal/assets/read/desc"
+	"github.com/ActiveMemory/ctx/internal/config/embed/text"
 	errMcp "github.com/ActiveMemory/ctx/internal/err/mcp"
 	ctxIo "github.com/ActiveMemory/ctx/internal/io"
 	"github.com/ActiveMemory/ctx/internal/rc"
@@ -60,7 +62,9 @@ func (h *Handler) SteeringGet(prompt string) (string, error) {
 
 	var sb strings.Builder
 	for _, sf := range filtered {
-		fmt.Fprintf(&sb, "## %s\n\n%s\n\n", sf.Name, sf.Body)
+		fmt.Fprintf(&sb,
+			desc.Text(text.DescKeyMCPSteeringSection),
+			sf.Name, sf.Body)
 	}
 
 	return sb.String(), nil
@@ -105,14 +109,18 @@ func (h *Handler) Search(query string) (string, error) {
 			lineNum++
 			line := scanner.Text()
 			if strings.Contains(strings.ToLower(line), queryLower) {
-				fmt.Fprintf(&sb, "%s:%d: %s\n", e.Name(), lineNum, line)
+				fmt.Fprintf(&sb,
+					desc.Text(text.DescKeyMCPSearchHitLine),
+					e.Name(), lineNum, line)
 				matches++
 			}
 		}
 	}
 
 	if matches == 0 {
-		return fmt.Sprintf("No matches for %q in %s.", query, h.ContextDir), nil
+		return fmt.Sprintf(
+			desc.Text(text.DescKeyMCPSearchNoMatch),
+			query, h.ContextDir), nil
 	}
 
 	return sb.String(), nil