Skip to content

fix: correct misleading FileNotFoundError message in example#1004

Merged
VaibhavUpreti merged 1 commit intoTracer-Cloud:mainfrom
blut-agent:main
Apr 27, 2026
Merged

fix: correct misleading FileNotFoundError message in example#1004
VaibhavUpreti merged 1 commit intoTracer-Cloud:mainfrom
blut-agent:main

Conversation

@blut-agent
Copy link
Copy Markdown
Contributor

Summary

Fixed the misleading FileNotFoundError message in the example code. Changed "empty file not present" to "input file not present" to correctly match the os.path.exists() condition being checked.

Related

Closes #828

The condition checks os.path.exists() but the error message says
'empty file' — should say 'input file' to match the missing-file
condition.

Closes Tracer-Cloud#828
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 27, 2026

Greptile Summary

This PR corrects a one-line documentation fix in tests/AGENTS.md, changing the example FileNotFoundError message from "empty file not present" to "input file not present" to accurately reflect the os.path.exists() check being illustrated.

Confidence Score: 5/5

Safe to merge — documentation-only fix with no code or logic changes.

The change is a single-word correction in an example snippet inside a markdown file. It introduces no runtime behaviour changes and correctly aligns the error message with the condition it describes. All findings are P2 or lower.

No files require special attention.

Important Files Changed

Filename Overview
tests/AGENTS.md Single-line correction to example error message — "empty file not present" → "input file not present" — making the message consistent with the os.path.exists() check in the code snippet.

Sequence Diagram

sequenceDiagram
    participant Caller
    participant use_case as use_case.py
    participant OS as os.path

    Caller->>use_case: extract_and_validate(input_path)
    use_case->>OS: exists(input_path)
    OS-->>use_case: False
    use_case-->>Caller: raise FileNotFoundError("input file not present: {input_path}")
Loading

Reviews (1): Last reviewed commit: "fix: correct misleading FileNotFoundErro..." | Re-trigger Greptile

@hamzzaaamalik
Copy link
Copy Markdown
Collaborator

Spotted that #1002 is fixing the same line in the same file for issue #828. Both fixes are correct. Wording-wise I lean toward #1002's "not found" since it matches the FileNotFoundError convention, but happy to defer. cc @davincios / @VaibhavUpreti for which to merge.

Copy link
Copy Markdown
Member

@VaibhavUpreti VaibhavUpreti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome @blut-agent !

@VaibhavUpreti VaibhavUpreti merged commit d82732a into Tracer-Cloud:main Apr 27, 2026
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.

Fix the misleading FileNotFoundError example in tests/AGENTS.md

3 participants