Skip to content

fix: store base content for deleted PR files#22

Closed
CaptainTimon wants to merge 1 commit intoentrius:testfrom
CaptainTimon:fix/deleted-pr-file-base-content
Closed

fix: store base content for deleted PR files#22
CaptainTimon wants to merge 1 commit intoentrius:testfrom
CaptainTimon:fix/deleted-pr-file-base-content

Conversation

@CaptainTimon
Copy link
Copy Markdown

@CaptainTimon CaptainTimon commented May 8, 2026

Summary

  • Store base-side content for deleted PR files instead of treating removed files as already complete.
  • Avoid head-side blob lookups for removed files, so deleted files persist base_content while head_content remains null.
  • Add focused regression coverage for removed-file content batching without changing package scripts, dependencies, or build behavior.

Related Issues

Fixes #20

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other (describe below)

Testing

  • git diff --check
  • node --test -r ts-node/register src/webhook/github-fetcher.service.spec.ts
  • npm run format:check
  • npm run lint
  • npm run build

Checklist

  • I have read the Contributing Guide
  • Code builds without errors
  • New and existing tests pass (if applicable)
  • Documentation updated (if applicable)
  • No unnecessary dependencies added

@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label May 8, 2026
@CaptainTimon CaptainTimon changed the title fix: fetch base content for deleted PR files fix: store base content for deleted PR files May 8, 2026
@CaptainTimon CaptainTimon force-pushed the fix/deleted-pr-file-base-content branch from b5644d0 to 201d533 Compare May 8, 2026 21:51
@CaptainTimon CaptainTimon force-pushed the fix/deleted-pr-file-base-content branch from 201d533 to d159f4b Compare May 8, 2026 21:57
@anderdc anderdc closed this May 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Deleted PR files are marked complete without base content

2 participants