fix: add before/after logging for all state-mutating operations

[tsavo-at-pieces]

Summary

Comprehensive audit found 8 silent mutations and 6 missing-BEFORE-state operations across the CLI commands.

Critical Silent Mutations (no logging at all)

Location	Operation
create_release_command.dart:238-265	git config + git add (verbose-only)
manage_cicd.dart:2019-2039	git config + git add (verbose-only)
act.dart:101-105	Game plan JSON writes
consumers_command.dart:1214-1224	_atomicWriteJson
step_summary.dart:12	GitHub Step Summary appends
template_manifest.dart:96	Template version tracker save
version_detection.dart:148,163	Gemini response JSON writes
sub_package_utils.dart:278,369	Sub-package pubspec mutations

Missing BEFORE State

Location	Operation
create_release_command.dart:117	pubspec version bump (no old→new)
update_command.dart:219-256	Template file copies (no diff)
audit_command.dart:126	pubspec fixes (no before state)
compose_command.dart:116	CHANGELOG write
autodoc_command.dart:59-62	Legacy migration
init_command.dart:242-249	.gitignore append

Pattern to Follow

Every mutation should:

1. Before: Log intent + current state
2. Execute: Do the thing
3. After: Log success/failure + what changed

Fix Plan

• Add logging wrappers for git operations in release commands
• Add before/after logging for act.dart game plan writes
• Log old→new version in pubspec bumps
• Log file diffs or summaries for template updates
• Ensure _atomicWriteJson logs its operations
• Add logging for all writeAsStringSync calls in release pipeline

[tsavo-at-pieces]

Partial fix in PR #28

Addressed the highest-impact logging gaps:

• writeJson() / readJson() in json_schemas.dart — shared utility now logs file name and key count on every write, and warns on read failures (covers 12+ callsites across all triage phases)
• create_release_command.dart — added logging for git config identity setup, file staging count, and authenticated remote URL setup
• manage_cicd.dart — added logging for git config and staging in legacy release flow
• sub_package_utils.dart — upgraded convertSiblingDepsForRelease() logging from verbose-only to always-on for pubspec writes and resolution stripping; added Logger.warn() for failed resolution strip

Remaining gaps (lower priority, can be follow-up):

• consumers_command.dart — 5 writeAsStringSync calls for temp/output files (discovery snapshots, release metadata)
• create_release_command.dart — release artifact writes (fallback notes, linked_issues.json, changelog_entry.md, contributors.json) already have surrounding context logging but no explicit per-file logging