[spec-extractor] Update package specifications for gitutil, logger, stringutil, timeutil

[github-actions]

Package Specification Updates

This PR fixes inaccuracies found during the spec-extractor round-robin run for 4 packages.

Package	Status	Changes
gitutil	Updated	Added missing IsValidFullSHA; fixed IsRateLimitError description
logger	Updated	Documented ACTIONS_RUNNER_DEBUG=true behavior
stringutil	Updated	Fixed 3 inaccurate descriptions
timeutil	Updated	Fixed incorrect FormatDurationNs example

What Changed

pkg/gitutil/README.md

• IsRateLimitError: description now includes all 3 matched patterns ("api rate limit exceeded", "rate limit exceeded", "secondary rate limit") — the "secondary rate limit" case was previously undocumented
• IsValidFullSHA: added missing documentation for this exported function (validates 40-character lowercase hex SHAs)

pkg/logger/README.md

• Added documentation for ACTIONS_RUNNER_DEBUG=true behavior: when this env var is set (as in GitHub Actions debug runs) and DEBUG is not explicitly set, all loggers are enabled automatically (equivalent to DEBUG=*)

pkg/stringutil/README.md

• NormalizeWhitespace: previous description said it "collapses multiple consecutive whitespace characters into a single space" — this was completely wrong. The function trims trailing whitespace from each line and ensures exactly one trailing newline
• IsPositiveInteger: previous description said "non-empty string containing only digit characters" — this was incomplete. The function actually requires strictly positive integers (no zero, no leading zeros, must parse to > 0)
• NormalizeSafeOutputIdentifier: previous description said it only converts dashes to underscores — it also converts periods (required because MCP tool names cannot contain periods)

pkg/timeutil/README.md

• FormatDurationNs example: FormatDurationNs(2_500_000_000) documented as returning "2s" but Go's time.Duration.Round(time.Second) rounds half-away from zero, so 2.5s rounds to "3s". Fixed example to use 2_000_000_000 (exactly 2 billion nanoseconds = 2s)

Extraction Method

• Source code analysis of exported function implementations and their doc comments
• Verification of all examples against actual implementation logic
• Comparison of spec_test.go files (which already noted SPEC_MISMATCH for the timeutil issue)

Round-Robin State

• Packages processed this run: gitutil, logger, stringutil, timeutil (indices 6, 7, 12, 15)
• Next packages in rotation: tty, types, typeutil, workflow (indices 16+)
• Total packages: 21
• Coverage: First run — cache initialized

---

Auto-generated by Package Specification Extractor workflow §24717864418

Generated by Package Specification Extractor · ● 4.9M · ◷

• expires on Apr 24, 2026, 10:49 AM UTC