Bump github.com/senzing-garage/go-helpers from 0.6.14 to 0.6.15

[dependabot]

Bumps github.com/senzing-garage/go-helpers from 0.6.14 to 0.6.15.

Release notes

Sourced from github.com/senzing-garage/go-helpers's releases.

0.6.15

See CHANGELOG.md

Changelog

Sourced from github.com/senzing-garage/go-helpers's changelog.

[0.6.15] - 2026-01-07

Changed in 0.6.15

• Regenerated test certificates in testdata/certificates for 2026

Commits

• 379106e #189 Update certificates for 2026 (#190)
• See full diff in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

[senzingdevops]

Automated: approving this pull request because it includes a patch update

[github-actions]

🤖 Claude Code Review

Code Review Report

Summary

This is a dependency version bump pull request that upgrades github.com/senzing-garage/go-helpers from v0.6.14 to v0.6.15. This is a routine maintenance change updating a minor/patch version of a dependency.

---

Review Checklist Results

Code Quality

✅ Code follows style guide - No code changes to evaluate; dependency bumps don't introduce style issues.

✅ No commented-out code - No code changes present.

✅ Meaningful variable names - N/A (no new code).

✅ DRY principle followed - N/A (no new code).

✅ Identify Defects - No bugs, logic errors, or defects introduced. This is a straightforward dependency version update in go.mod and go.sum.

✅ Project memory configuration (.claude/CLAUDE.md) - Reviewed the project instructions; they are general and not specific to local development environments. No issues found.

Testing

❌ Unit tests for new functions - Not applicable for dependency bumps, but no tests were verified to still pass. The checklist requires test coverage verification, which isn't shown in this PR.

⚠️ Integration tests for new endpoints - N/A (no new endpoints).

⚠️ Edge cases covered - N/A (no new functionality).

❌ Test coverage > 80% - Not verified in this PR. According to CLAUDE.md (go.mod:0), the project requires 75% coverage threshold, but this PR doesn't show test execution results.

Documentation

❌ CHANGELOG.md updated - The CHANGELOG.md (line 8-10) has an empty [Unreleased] section. This dependency bump should be documented there with an entry like:

## [Unreleased]

### Changed

- Upgraded github.com/senzing-garage/go-helpers from 0.6.14 to 0.6.15

✅ Readme updated if needed - N/A (dependency bumps don't typically require README updates).

✅ API docs updated - N/A (no API changes).

✅ Inline comments for complex logic - N/A (no new code).

✅ Markdown files follow CommonMark - Existing markdown appears properly formatted.

Security

✅ No hardcoded credentials - None present.

✅ Input validation implemented - N/A (no new code).

✅ Proper error handling - N/A (no new code).

✅ No sensitive data in logs - N/A (no new code).

✅ No license files (.lic) or AQAAAD strings - None detected in the diff.

---

Required Actions

1. Update CHANGELOG.md (CRITICAL) - Add an entry in the [Unreleased] section documenting this dependency upgrade (CHANGELOG.md:8-10).

2. Verify tests pass (RECOMMENDED) - Run make test to ensure the dependency upgrade doesn't break existing functionality.

3. Verify coverage (RECOMMENDED) - Run make check-coverage to ensure the 75% coverage threshold is still met.

---

Overall Assessment

This is a clean, low-risk dependency version bump. The only blocking issue is the missing CHANGELOG.md entry. After adding that entry and verifying tests pass, this PR should be safe to merge.

Automated code review analyzing defects and coding standards

[senzingdevops]

Automated: approving this pull request because it includes a patch update

[github-actions]

🤖 Claude Code Review

Code Review - PR Analysis

Summary

This PR makes dependency updates and reduces coverage/linter requirements. The changes are primarily maintenance-oriented with some concerning quality standard reductions.

---

Code Quality

✅ Code follows style guide

No actual code changes to evaluate. Dependency updates and configuration changes only.

✅ No commented-out code

No commented-out code added.

✅ Meaningful variable names

N/A - no code changes.

✅ DRY principle followed

N/A - no code changes.

✅ Identify Defects

No bugs or defects introduced. Dependency updates appear routine.

⚠️ Project Memory Configuration (./.claude/CLAUDE.md)

The project memory configuration exists and appears appropriate. However, note:

• Documentation states coverage threshold is 75% but this PR reduces it to 70%
• .claude/CLAUDE.md:79 needs updating to reflect new 70% threshold

File: .github/coverage/testcoverage.yaml:18-26
Issue: Coverage thresholds reduced from 75% to 70% without documented justification.

---

Testing

❌ Test coverage > 80%

CRITICAL CONCERN: Coverage threshold was reduced from 75% to 70% across all metrics (file, package, total).

Files affected:

• .github/coverage/testcoverage.yaml:18 (file threshold)
• .github/coverage/testcoverage.yaml:22 (package threshold)
• .github/coverage/testcoverage.yaml:26 (total threshold)

Why this is concerning:

• Lowering quality standards typically indicates difficulty meeting existing requirements
• The project documentation (.claude/CLAUDE.md:79) explicitly states "Coverage threshold: 75%"
• No justification or explanation provided in the PR
• This contradicts the checklist requirement of "> 80%" coverage

Recommendation: Provide justification for the reduction or revert to 75% threshold.

---

Documentation

❌ CHANGELOG.md updated

No CHANGELOG.md update included. For dependency updates and configuration changes affecting quality thresholds, changelog entries are expected.

❌ Documentation accuracy

The .claude/CLAUDE.md file states coverage threshold is 75% but this PR changes it to 70%. Documentation must be updated.

File to update: .claude/CLAUDE.md:79
Change from: Coverage threshold: 75% (file, package, total)
To: Coverage threshold: 70% (file, package, total)

✅ Markdown formatting

N/A - no markdown files modified.

---

Security

✅ No hardcoded credentials

No credentials detected.

✅ No sensitive data in logs

N/A - configuration changes only.

✅ No license files checked in

No .lic files or AQAAAD strings detected in the diff.

✅ Input validation & error handling

N/A - no code changes.

---

Additional Observations

Linter Configuration Change

File: .github/linters/.golangci.yaml:113

The prealloc linter was disabled. This linter suggests preallocating slices when the size is known, which can improve performance.

Impact: Minor performance optimizations may be missed going forward. This change reduces code quality enforcement.

---

Critical Issues Summary

1. Coverage threshold reduced without justification - Requires explanation or reversion
2. Documentation mismatch - .claude/CLAUDE.md needs updating to reflect 70% threshold
3. No CHANGELOG entry - Should document both dependency updates and configuration changes
4. Linter weakening - prealloc disabled, reducing performance optimization enforcement

---

Recommendations

1. Add CHANGELOG.md entry documenting:

   • Dependency updates (go-helpers 0.6.14→0.6.15, sz-sdk-go 0.15.11→0.15.12)
   • Coverage threshold reduction (75%→70%)
   • Linter configuration change (prealloc disabled)

2. Update .claude/CLAUDE.md:79 to reflect new 70% coverage threshold

3. Provide justification for reducing quality standards, or consider reverting coverage and linter changes

4. Consider whether 70% coverage meets project quality goals, especially given the checklist requires "> 80%"

---

Automated code review analyzing defects and coding standards