Add Operator() method to Signals

[clouvet]

Summary

• Adds an Operator() method on Signals that returns a single classification string
• Precedence: ci > agent > interactive > unknown
• Convenience for consumers that need one label without reimplementing the precedence logic
• Raw fields (CI, Agent, Interactive) remain available for finer-grained use

Usage

s := clientsignals.DetectOnce()
fmt.Println(s.Operator())    // "ci", "agent", "interactive", or "unknown"
fmt.Println(s.Agent)          // e.g. "claude-code" (when Operator() == "agent")

Motivation

flyctl metrics needs to classify deploy/launch traffic by operator type. Rather than each consumer reimplementing the same precedence switch, this centralizes it in the library.

Related PRs

• Add operator/agent_name to deploy/launch metrics payloads flyctl#4995 — will use s.Operator() instead of its own OperatorFromSignals helper
• superfly/flyctl-metrics#67 — consumes the operator label on the metrics server side

Test plan

• go test ./... passes
• Table-driven tests cover all precedence cases (ci wins over agent, agent wins over interactive, unknown fallback)

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR adds a small convenience API to the Go clientsignals package: a Signals.Operator() method that collapses the existing CI, Agent, and Interactive signals into a single stable classification label with a defined precedence order. This helps downstream consumers avoid re-implementing the same precedence logic while still leaving the raw signal fields available for more detailed use.

Changes:

• Added Signals.Operator() returning one of: "ci", "agent", "interactive", "unknown" with precedence ci > agent > interactive > unknown.
• Added a table-driven unit test covering all precedence branches and the default fallback.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
go/signals.go	Adds Operator() method to map existing signals into a single operator classification string with documented precedence.
go/signals_test.go	Adds table-driven tests validating Operator() precedence and fallback behavior.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.