Skip to content

chore: gitignore tmp/#1780

Merged
senamakel merged 3 commits into
tinyhumansai:mainfrom
senamakel:chore/gitignore-tmp
May 15, 2026
Merged

chore: gitignore tmp/#1780
senamakel merged 3 commits into
tinyhumansai:mainfrom
senamakel:chore/gitignore-tmp

Conversation

@senamakel
Copy link
Copy Markdown
Member

@senamakel senamakel commented May 15, 2026

Summary

  • Add tmp/ to .gitignore so local scratch output isn't tracked.

Problem

  • A top-level tmp/ directory has been used for local scratch artifacts and keeps showing up as untracked in git status, creating noise and risk of accidentally committing local-only output.

Solution

  • Append tmp/ to the repo .gitignore.

Submission Checklist

  • N/A: gitignore-only change, no code paths exercised.
  • N/A: gitignore-only change, no diff coverage applicable.
  • N/A: no feature surface added/removed/renamed.
  • N/A: no feature IDs affected.
  • N/A: no new network dependencies.
  • N/A: does not touch release-cut surfaces.
  • N/A: no linked issue.

Impact

  • None at runtime. Reduces VCS noise from local scratch artifacts.

Related

  • Closes:
  • Follow-up PR(s)/TODOs:

AI Authored PR Metadata (required for Codex/Linear PRs)

Linear Issue

  • Key: N/A
  • URL: N/A

Commit & Branch

  • Branch: chore/gitignore-tmp
  • Commit SHA: a31bab0

Validation Run

  • N/A: gitignore-only change
  • N/A: gitignore-only change
  • Focused tests: N/A
  • Rust fmt/check (if changed): N/A
  • Tauri fmt/check (if changed): N/A

Validation Blocked

  • command: N/A
  • error: N/A
  • impact: N/A

Behavior Changes

  • Intended behavior change: ignore tmp/ directory in git
  • User-visible effect: none

Parity Contract

  • Legacy behavior preserved: yes
  • Guard/fallback/dispatch parity checks: N/A

Duplicate / Superseded PR Handling

  • Duplicate PR(s): none
  • Canonical PR: this PR
  • Resolution: N/A

Summary by CodeRabbit

  • Chores

    • Ignore temporary build files (tmp/) in repository.
  • New Features

    • Added a test-planning CLI to fetch GitHub issues/PRs and synthesize them into an aggregated test map and human-readable report.
  • Documentation

    • Added usage guide explaining the test-planning pipeline, CLI phases, outputs, examples for running/resuming, and configuration notes.

Review Change Stack

@senamakel senamakel requested a review from a team May 15, 2026 04:28
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 15, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 09a62971-8c7a-4376-a0b8-a94ddcf2b0d4

📥 Commits

Reviewing files that changed from the base of the PR and between a31bab0 and 88bb2b4.

📒 Files selected for processing (3)
  • .gitignore
  • scripts/test-planning/README.md
  • scripts/test-planning/build-github-test-map.mjs

📝 Walkthrough

Walkthrough

Adds a new test-planning CLI and README under scripts/test-planning/ to fetch GitHub issues and PRs, synthesize them with an LLM into canonical features/test-map outputs (JSONL and Markdown), persist batch and manifest artifacts, and also updates .gitignore to ignore tmp/.

Changes

Test-planning pipeline

Layer / File(s) Summary
CLI entrypoint and helpers
scripts/test-planning/build-github-test-map.mjs
Argument parsing, usage/help text, timestamps, and process/filesystem helpers for the CLI.
GraphQL normalization & paginated fetch
scripts/test-planning/build-github-test-map.mjs
Normalize GitHub issue/PR GraphQL nodes to a unified item shape and implement paginated fetch with updated-time filtering and caps.
JSON/JSONL I/O and fetchPhase
scripts/test-planning/build-github-test-map.mjs
Utilities to read/write JSON/JSONL and the fetchPhase that writes raw-issues.jsonl, raw-prs.jsonl, raw-all.jsonl, and manifest.json.
Batching & prompt construction
scripts/test-planning/build-github-test-map.mjs, scripts/test-planning/README.md
Chunk raw items and build LLM synthesis prompts with per-item formatting; README documents usage and outputs.
LLM schema & invocation adapters
scripts/test-planning/build-github-test-map.mjs
Defines expected LLM JSON schema and adapters to invoke Codex or Claude CLIs and parse structured responses.
Feature merge, sort & render
scripts/test-planning/build-github-test-map.mjs
Deduplicate/merge LLM outputs into canonical features, sort by priority, and render test-map.jsonl and test-map.md.
Main orchestration
scripts/test-planning/build-github-test-map.mjs
Top-level main() wiring to run fetch and/or synthesize phases and persist results.

Temporary Directory Ignore Configuration

Layer / File(s) Summary
Temporary directory ignore configuration
.gitignore
Add tmp/ to .gitignore to prevent temporary files under tmp/ from being tracked.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

Poem

🐰
I fetched issues by moonlight’s hum,
Chatted with LLMs till ideas come,
Batches saved in a tidy row,
A test-map blooms where raw items grow,
Hop, scribble, stash — the pipeline’s done.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: gitignore tmp/' clearly and concisely summarizes the main change: adding tmp/ to .gitignore to reduce VCS noise from local scratch artifacts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.


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

coderabbitai[bot]
coderabbitai Bot previously approved these changes May 15, 2026
@senamakel senamakel self-assigned this May 15, 2026
@senamakel senamakel merged commit 2702e25 into tinyhumansai:main May 15, 2026
17 of 19 checks passed
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