Silent amplitude logger

[ivpusic]

Currently it shows debug statements, in release build

[cursor]

PR Summary

Low Risk
Behavior change is limited to telemetry client logging and developer build/run targets, with minimal impact on core CLI functionality.

Overview
Suppresses noisy Amplitude SDK logging in non-dev CLI builds by wiring a silentLogger into the Amplitude client config unless CLIVersion is empty/dev.

Updates the makefile to split dune-cli into a fast dune-cli-quick build (no lint), add a run target that executes the locally built binary with ARGS, and keep build/dune-cli running lint by default.

^{Written by Cursor Bugbot for commit 87090ed. Configure here.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c5b0f86e-07af-4528-bea0-4d228406fcb5

📥 Commits

Reviewing files that changed from the base of the PR and between 87090ed and 10352b6.

📒 Files selected for processing (1)

• makefile

---

📝 Walkthrough

Walkthrough

Makefile: updated PHONY targets, removed install and dune-cli blocks, replaced lint dependency mapping and added a run target that executes the CLI via go run. tracking/tracking.go: added dev-version detection and a no-op silentLogger to suppress Amplitude SDK logging in non-release builds.

Changes

Cohort / File(s)	Summary
Build Configuration makefile	Updated PHONY targets to include run and removed install. Removed the dune-cli target block and adjusted lint/build mappings; added run target that executes go run -ldflags '$(LDFLAGS)' ./cmd $(ARGS).
Logging & Observability tracking/tracking.go	Added isDevVersion helper and conditional logic to set Amplitude SDK logger to a silentLogger (no-op) when CLI version is non-release/dev; introduced silentLogger implementing logger methods as no-ops.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Silent amplitude logger' clearly summarizes the main change: suppressing amplitude SDK logger output in release builds.
Description check	✅ Passed	The description 'Currently it shows debug statements, in release build' is related to the changeset and identifies the problem being addressed.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}