Skip to content

Comments

more error reporter fixes for github action#271

Merged
kfirstri merged 1 commit intomainfrom
more-error-reporter-fixes
Feb 18, 2026
Merged

more error reporter fixes for github action#271
kfirstri merged 1 commit intomainfrom
more-error-reporter-fixes

Conversation

@kfirstri
Copy link
Collaborator

@kfirstri kfirstri commented Feb 18, 2026

Note

Description

This PR improves the reliability and output quality of the daily error report GitHub Actions workflow. It fixes issues with large issue bodies being rejected by gh issue create, reduces the exception_list substring limit to avoid token bloat, and adds clearer instructions for the Claude agent reading the data files.

Related Issue

None

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Other (please describe):

Changes Made

  • Reduced exception_list substring truncation from 3000 to 1500 characters to reduce token usage in the AI analysis step
  • Changed issue creation to a two-step process: first write the body to error-report-body.md, then use --body-file flag with gh issue create to avoid inline body size limits
  • Added Write to the list of allowed tools in claude_args so the agent can write the body file
  • Added explicit instruction to use cat via Bash tool (not the Read tool) to read data files, preventing token limit issues
  • Added instruction requiring the agent to read ALL rows from both JSONL files for complete coverage
  • Updated the stack trace truncation note in the prompt to reflect the new 1500-char limit

Testing

  • I have tested these changes locally
  • I have added/updated tests as needed
  • All tests pass (npm test)

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have updated docs/ (AGENTS.md) if I made architectural changes

Additional Notes

All changes are confined to the .github/workflows/daily-error-report.yml workflow file. The two-step issue creation approach (write body to file, then --body-file) is necessary because large markdown bodies exceed the shell argument size limit when passed inline to gh issue create.


🤖 Generated by Claude | 2026-02-18 23:55 UTC

@kfirstri kfirstri self-assigned this Feb 18, 2026
@github-actions
Copy link
Contributor

🚀 Package Preview Available!


Install this PR's preview build with npm:

npm i @base44-preview/cli@0.0.32-pr.271.5d60e7b

Prefer not to change any import paths? Install using npm alias so your code still imports base44:

npm i "base44@npm:@base44-preview/cli@0.0.32-pr.271.5d60e7b"

Or add it to your package.json dependencies:

{
  "dependencies": {
    "base44": "npm:@base44-preview/cli@0.0.32-pr.271.5d60e7b"
  }
}

Preview published to npm registry — try new features instantly!

@kfirstri kfirstri merged commit 8a812e0 into main Feb 18, 2026
7 checks passed
@kfirstri kfirstri deleted the more-error-reporter-fixes branch February 18, 2026 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant