Skip to content
This repository was archived by the owner on Apr 21, 2026. It is now read-only.

chore: gitignore .forge runtime artifacts (telemetry, baseline)#145

Merged
LucasSantana-Dev merged 1 commit into
mainfrom
chore/gitignore-forge-runtime
Mar 15, 2026
Merged

chore: gitignore .forge runtime artifacts (telemetry, baseline)#145
LucasSantana-Dev merged 1 commit into
mainfrom
chore/gitignore-forge-runtime

Conversation

@LucasSantana-Dev
Copy link
Copy Markdown
Member

@LucasSantana-Dev LucasSantana-Dev commented Mar 15, 2026

Summary

.forge/test-autogen-telemetry.jsonl and .forge/test-autogen-baseline.json are generated by the forge-ai-init pre-commit/pre-push hooks (landed in PR #144). They are ephemeral run-data files and should not be tracked in version control.

Changes

  • .gitignore: add two .forge/ runtime artifact patterns under a new # Forge AI runtime artifacts section

Summary by CodeRabbit

  • Chores
    • Updated repository configuration to exclude test artifacts from version control.

test-autogen-telemetry.jsonl and test-autogen-baseline.json are generated
by the forge-ai-init pre-commit/pre-push hooks. They are ephemeral run data
and should not be tracked in version control.
@LucasSantana-Dev LucasSantana-Dev requested a review from a team as a code owner March 15, 2026 15:43
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 15, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Two Forge AI runtime artifact entries (test-autogen-telemetry.jsonl and test-autogen-baseline.json) were added to the .gitignore file under the existing OpenCode config section. This prevents these generated test artifacts from being tracked in version control.

Changes

Cohort / File(s) Summary
Gitignore Configuration
.gitignore
Added two Forge AI test artifact entries: .forge/test-autogen-telemetry.jsonl and .forge/test-autogen-baseline.json

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hidden treasures, tucked away safe,
Forge artifacts no longer in the race,
.gitignore grows with whisker'd grace,
Test logs and baselines find their place,
A rabbit's work, clean and in place! 🐇✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: gitignore .forge runtime artifacts (telemetry, baseline)' directly and clearly describes the main change: adding Forge AI runtime artifact entries to .gitignore for telemetry and baseline files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/gitignore-forge-runtime
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

Forge AI Quality Gate

Passed — Score 100/100 (A+) · Delta +0 · 0 new findings

Category scores
Category Score
test-autogen 100

Powered by Forge AI Action · forgespace.co

@github-actions
Copy link
Copy Markdown

Project Scorecard


Scorecard: 84/100 (B)
────────────────────────────────────────
  security: 100/100 (A)
  quality: 80/100 (B) — 1 violations
  performance: 67/100 (D) — 1 violations
  compliance: 75/100 (C) — 1 violations
  dependency: 100/100 (A)

Recommendations:
  - Increase test coverage to meet the 80% threshold
  - Extend log retention to at least 90 days for compliance

@sonarqubecloud
Copy link
Copy Markdown

@LucasSantana-Dev LucasSantana-Dev merged commit 17e6dd5 into main Mar 15, 2026
24 of 25 checks passed
LucasSantana-Dev added a commit that referenced this pull request Mar 15, 2026
CHANGELOG:
- CI: actions/checkout + setup-node v4→v6 across 13 workflow files (#143)
- CI: Tenant profile inlined to .forge/, eliminated cross-repo PAT (#142)
- CI: Git hooks auto-detect tenant profile, eliminates ausentes warning (#144)
- chore: UIForge→Forge Space rebrand in scripts/patterns/memories (#141)
- chore: gitignore .forge runtime artifacts (#145)
- fix: knip.json 8 stale ignores removed, 0 issues
LucasSantana-Dev added a commit that referenced this pull request Mar 15, 2026
…148)

CHANGELOG:
- CI: actions/checkout + setup-node v4→v6 across 13 workflow files (#143)
- CI: Tenant profile inlined to .forge/, eliminated cross-repo PAT (#142)
- CI: Git hooks auto-detect tenant profile, eliminates ausentes warning (#144)
- chore: UIForge→Forge Space rebrand in scripts/patterns/memories (#141)
- chore: gitignore .forge runtime artifacts (#145)
- fix: knip.json 8 stale ignores removed, 0 issues
@LucasSantana-Dev LucasSantana-Dev deleted the chore/gitignore-forge-runtime branch March 16, 2026 16:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant