[Issue #8] Fix gitea skill documentation for PR diff command #14

HugoNijhuis · 2026-01-01T18:00:46Z

HugoNijhuis commented

2026-01-01 18:00:46 +00:00

Summary

Updated gitea skill to remove incorrect tea pulls <number> -f diff command
Documented correct method: tea pulls checkout + git diff main...HEAD
Updated code-reviewer agent to use the correct diff method

The tea CLI's -f diff flag only returns a URL to the diff, not the actual diff content. There's no tea command to output diff content directly.

Test plan

Verify documentation is accurate
Run code-reviewer agent to confirm it uses the correct commands

Closes #8

🤖 Generated with Claude Code

## Summary - Updated gitea skill to remove incorrect `tea pulls <number> -f diff` command - Documented correct method: `tea pulls checkout` + `git diff main...HEAD` - Updated code-reviewer agent to use the correct diff method The tea CLI's `-f diff` flag only returns a URL to the diff, not the actual diff content. There's no tea command to output diff content directly. ## Test plan - [ ] Verify documentation is accurate - [ ] Run code-reviewer agent to confirm it uses the correct commands Closes #8 🤖 Generated with [Claude Code](https://claude.com/claude-code)

HugoNijhuis added 1 commit 2026-01-01 18:00:47 +00:00

Fix PR diff documentation to use git instead of tea c815f2ae6f

The tea CLI doesn't have a command to output PR diff content directly.
The -f diff flag only returns a URL. Updated docs to use tea pulls
checkout followed by git diff main...HEAD.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

HugoNijhuis commented

2026-01-01 18:01:21 +00:00

AI Code Review

This is an automated review generated by the code-reviewer agent.

Summary

Documentation fix correctly replaces the non-functional tea pulls -f diff command with the proper two-step approach using tea pulls checkout + git diff. The changes are accurate and improve both the gitea skill documentation and code-reviewer agent instructions.

Findings

Code Quality

Changes are well-structured and clear
Documentation follows existing formatting conventions
The comment explaining why tea doesn't have a direct diff command adds valuable context

Potential Bugs

No issues found - the documented commands are correct and functional

Security Concerns

No security concerns identified

Style Notes

Consistent with codebase documentation style
Good use of inline comments to explain the limitation

Test Coverage

The PR description includes a test plan to verify the documentation and agent behavior
Consider manually testing the code-reviewer agent with this PR to validate the new approach works

Verdict

LGTM

This is a straightforward documentation fix that addresses a real issue (the -f diff flag returns a URL, not content). The corrected approach using checkout + git diff is the proper method for retrieving PR diffs with tea CLI. Ready to merge once the test plan checklist is completed.

## AI Code Review > This is an automated review generated by the code-reviewer agent. ### Summary Documentation fix correctly replaces the non-functional `tea pulls -f diff` command with the proper two-step approach using `tea pulls checkout` + `git diff`. The changes are accurate and improve both the gitea skill documentation and code-reviewer agent instructions. ### Findings #### Code Quality - Changes are well-structured and clear - Documentation follows existing formatting conventions - The comment explaining why tea doesn't have a direct diff command adds valuable context #### Potential Bugs - No issues found - the documented commands are correct and functional #### Security Concerns - No security concerns identified #### Style Notes - Consistent with codebase documentation style - Good use of inline comments to explain the limitation #### Test Coverage - The PR description includes a test plan to verify the documentation and agent behavior - Consider manually testing the code-reviewer agent with this PR to validate the new approach works ### Verdict **LGTM** This is a straightforward documentation fix that addresses a real issue (the -f diff flag returns a URL, not content). The corrected approach using checkout + git diff is the proper method for retrieving PR diffs with tea CLI. Ready to merge once the test plan checklist is completed.

HugoNijhuis merged commit c815f2ae6f into main

2026-01-01 18:03:55 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: flowmade-one/architecture#14