Skip to content

Handle @ in local marketplace paths - #34959

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/aa55c8cd9838908e15b6a17f8d6058e95aa13ba5
Jul 23, 2026
Merged

Handle @ in local marketplace paths#34959
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/aa55c8cd9838908e15b6a17f8d6058e95aa13ba5

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 23, 2026

Copy link
Copy Markdown

Handle @ in local marketplace paths

Why

Relative local marketplace paths can contain @, but source parsing treated it as a Git ref separator.

What changed

  • Skip @ ref parsing when the marketplace source has local path syntax.
  • Cover adding a local marketplace from ./alice@example.com/marketplace through the app-server API.

## Why

Relative local marketplace paths can contain `@`, but source parsing treated it as a Git ref separator.

## What changed

- Skip `@` ref parsing when the marketplace source has local path syntax.
- Cover adding a local marketplace from `./alice@example.com/marketplace` through the app-server API.

GitOrigin-RevId: aa55c8cd9838908e15b6a17f8d6058e95aa13ba5
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/aa55c8cd9838908e15b6a17f8d6058e95aa13ba5 branch from f39937d to 7d748d3 Compare July 23, 2026 15:32
@copyberry
copyberry Bot merged commit 7d748d3 into main Jul 23, 2026
32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/aa55c8cd9838908e15b6a17f8d6058e95aa13ba5 branch July 23, 2026 15:33
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 23, 2026
@BaxLi
BaxLi temporarily deployed to issue-triage July 23, 2026 15:35 — with GitHub Actions Inactive
@BaxLi
BaxLi temporarily deployed to issue-triage July 23, 2026 15:35 — with GitHub Actions Inactive
@BaxLi
BaxLi temporarily deployed to issue-triage July 23, 2026 15:35 — with GitHub Actions Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants