[squad-plan] Fix error message compliance: compiler_pre_activation_job.go, tools_validation_github.go, evals_config.go, spec.go, add_package_ [Content truncated due to length]

[github-actions]

Objective

Bring the error messages in these 5 files up to the repo's error-message style guide.

Context

Issue #52163 asks to repair error messages in batches of 5 files max. The lint-error-messages workflow (see run https://github.com/github/gh-aw/actions/runs/31510578209/job/93862987189) flagged these files as low-compliance:

• pkg/workflow/compiler_pre_activation_job.go: 5/17 compliant (29%)
• pkg/workflow/tools_validation_github.go: 0/11 compliant (0%)
• pkg/workflow/evals_config.go: 3/14 compliant (21%)
• pkg/cli/spec.go: 6/17 compliant (35%)
• pkg/cli/add_package_manifest.go: 15/26 compliant (57%)

Implementation Guidance

1. Read .github/skills/error-messages/SKILL.md for the style guide before editing.
2. For each fmt.Errorf / NewValidationError call flagged non-compliant, rewrite the message using the template: [what's wrong]. [what's expected]. [example of correct usage].
3. Avoid standalone negative wording (invalid, cannot, must, failed) without pairing it with expected behavior and a concrete fix.
4. Use NewValidationError(field, value, reason, suggestion) for *_validation.go files where applicable; use fmt.Errorf with %w wrapping for operational errors, adding recovery guidance.
5. Keep changes surgical — do not alter error-handling logic or control flow, only message text.

Acceptance Criteria

• Running the lint-error-messages check locally (or via gh aw tooling used by that workflow) shows improved/100% compliance for these 5 files.
• make fmt run after changes.
• make test-unit (or targeted package tests for pkg/workflow and pkg/cli) passes.
• No behavior change — only error message text updated.

Dependency Notes

Independent of other batches; can be worked in parallel with the other 7 sub-issues from this plan.

Generated by 🧑‍🤝‍🧑 Squad Plan · auto · 73.9 AIC · ⌖ 8.23 AIC · ⊞ 9.3K · ◷
_{Comment /squad-plan to run again}