Skip to content

fix: avoid duplicate restoration of explicit directory markers - #330

Open
jacksonriding wants to merge 1 commit into
LibreChat-AI:mainfrom
jacksonriding:fix/avoid-duplicate-dirkeep-restoration
Open

jacksonriding wants to merge 1 commit into
LibreChat-AI:mainfrom
jacksonriding:fix/avoid-duplicate-dirkeep-restoration

Conversation

@jacksonriding

Copy link
Copy Markdown
Contributor

When LibreChat attaches a directory marker under a renamed display filename, the normalized object listing can return the same stored .dirkeep again under its original name. The sandbox then downloads both references to the stored path and rejects execution.

Suppress the extra listing entry by object ID within the listed session. Explicit downloads still preserve marker contents, and an identical object ID in another session does not suppress an inherited marker.

Validation on Bun 1.4.2:

  • All 36 egress gateway tests pass.
  • The new regression fails without the listing fix.
  • Service build completes, with three TypeScript warnings outside the modified listing logic.

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