[daily-compiler-quality] Daily Compiler Code Quality Report - 2026-08-17

[github-actions[bot]]

Compiler Code Quality Analysis Report

Analysis Date: 2026-08-17
Files Analyzed: compiler.go, compiler_jobs.go, compiler_safe_outputs_job.go
Overall Status: Some files need attention.

---

Executive Summary

We analyzed three compiler files. Each file has a test file. Each test file is large. Error handling looks solid in two files.

The compiler.go file has one large function. CompileWorkflowData is 186 lines long. This function should split into smaller helper functions.

compiler_safe_outputs_job.go is the largest file. It has 1065 lines and 31 functions. One function, buildJobLevelSafeOutputEnvVars, is 144 lines long.

---

Detailed File Analysis

1. compiler.go - Score: 74/100

Size: 605 lines, 8 functions
Test file: compiler_test.go (1095 lines)

• Structure: 18/25. The file mixes validation, YAML generation, and output writing.
• Readability: 13/20. CompileWorkflowData is 186 lines. This is too long.
• Error handling: 16/20. The file uses zero %w error wraps. Add error wrapping for clearer context.
• Testing: 18/20. Test file is large. Test-to-source ratio is about 1.8.
• Patterns: 9/15. Split large functions to improve reuse.

Top issues: one 186-line function; no error wrapping with %w.

2. compiler_jobs.go - Score: 80/100

Size: 703 lines, 24 functions
Test file: compiler_jobs_test.go (4511 lines)

• Structure: 21/25. Functions are short. Average length is near 25 lines.
• Readability: 17/20. Names are clear and descriptive.
• Error handling: 18/20. The file uses 20 %w error wraps. This is good practice.
• Testing: 19/20. Test-to-source ratio is about 6.4. This is very strong.
• Patterns: 12/15. buildJobs is 91 lines. Consider splitting.

Top issues: buildJobs function is 91 lines long.

3. compiler_safe_outputs_job.go - Score: 71/100

Size: 1065 lines, 31 functions
Test file: compiler_safe_outputs_job_test.go (1730 lines)

• Structure: 15/25. File is over 1000 lines. Split into smaller files.
• Readability: 13/20. buildJobLevelSafeOutputEnvVars is 144 lines long.
• Error handling: 14/20. The file uses only 5 %w error wraps for 31 functions.
• Testing: 16/20. Test-to-source ratio is about 1.6.
• Patterns: 13/15. Good naming and clear helper function separation.

Top issues: file size over 1000 lines; one function is 144 lines; low error-wrap ratio.

---

Summary Table

File	Score	Rating	Top Issue
compiler.go	74/100	Acceptable	One function is 186 lines long
compiler_jobs.go	80/100	Good	buildJobs function is 91 lines long
compiler_safe_outputs_job.go	71/100	Acceptable	File is 1065 lines; split it

Avg score: 75/100 · Files meeting threshold (≥75): 1/3

Top 3 Issues

1. CompileWorkflowData in compiler.go is 186 lines long. Split it into smaller functions.
2. compiler_safe_outputs_job.go has 1065 lines. Split it into two or three files.
3. compiler.go has zero error wraps with %w. Add error context to failures.

Recommended Action

Split CompileWorkflowData into smaller functions. This takes about 2 hours.

---

Cache Memory Summary

Cache Location: /tmp/gh-aw/cache-memory/compiler-quality/

• Total Files Tracked: 9
• Files Analyzed Today: 3 (compiler.go, compiler_jobs.go, compiler_safe_outputs_job.go)
• Files Remaining in Queue: 6 (compiler_activation_jobs.go, compiler_orchestrator.go, compiler_safe_outputs.go, compiler_safe_outputs_config.go, compiler_yaml.go, compiler_yaml_main_job.go)
• This is the first recorded analysis. No prior history exists for trend comparison.

Next run should prioritize the 6 remaining files, since they have never been analyzed.

---

Conclusion

The three files average 75/100. One file meets the quality threshold. The other two files need refactoring for size and error handling.

Next Steps:

1. Split CompileWorkflowData in compiler.go.
2. Split compiler_safe_outputs_job.go into smaller files.
3. Add %w error wrapping in compiler.go.

---

Report generated by Daily Compiler Quality Check workflow
Analysis powered by Serena MCP Server and static Go metrics
Cache memory: /tmp/gh-aw/cache-memory/compiler-quality/

Generated by 📊 Daily Compiler Quality Check · auto · 19.1 AIC · ⌖ 6.94 AIC · ⊞ 14.7K · ◷

• expires on Aug 17, 2026, 6:55 PM UTC-08:00

[github-actions[bot]]

This discussion has been marked as outdated by Daily Compiler Quality Check.

A newer discussion is available at Discussion #53563.