Add deb repository and remote pages

[warisshaikh1]

Addresses part of #277 (deb in the side menu).

Draft on purpose — I asked three scoping questions in #277 and haven't heard back yet. This is here so there's something concrete to react to rather than to pre-empt the answers. Happy to cut it down, extend it, or restructure it entirely.

What this adds

A Pulp deb menu section gated on hasPlugin('deb'), with Repositories and Remotes, mirroring Pulp file:

	list	detail	edit	tabs
remotes/deb/apt	yes	yes	yes	details
repositories/deb/apt	yes	yes	yes	details, versions, distributions

What it deliberately leaves out

• Publications. deb has two publication endpoints rather than a field: publications/deb/apt generates fresh metadata and needs a signing service to be consumable, while publications/deb/verbatim republishes upstream's Release/InRelease byte for byte so an ordinary ubuntu-archive-keyring verifies it. A publications tab has to either show both or ask which to create, which is a design decision rather than wiring, so it seemed better to agree it first.
• Content browsing. content/deb/packages is the useful one, but there are a dozen deb content endpoints where rpm has one. Easy follow-up once the shape here is agreed.

The one place deb genuinely differs from file

RemoteForm needed the APT fields, which have no equivalent elsewhere: distributions (suites), components, architectures, gpgkey, and the sync_sources / sync_udebs / sync_installer switches.

distributions is required, for deb only. Every other plugin's remote needs just a URL; pulp_deb rejects one without suites:

POST /pulp/api/v3/remotes/deb/apt/
{"name":"x","url":"http://deb.debian.org/debian/","tls_validation":true,"download_concurrency":10}
-> 400 {"distributions": ["This field is required."]}

So requiredFields becomes conditional. gpgkey reuses the FileUpload treatment ca_cert and client_cert already get, since it's an armoured key file rather than a one-line value.

Everything else is registry entries — plugin2api, the plugin unions on LazyRepositories / LazyDistributions / RepositoryForm, and deb-only optional fields on the shared RemoteType. All additive; ansible, container and file behaviour is unchanged (the RemoteForm diff is worth a look on that point specifically).

Testing

Verified against a live pulpcore 3.116.0 / pulp_deb 3.10.0 mirroring Ubuntu noble, with the dev server proxied at it (API_PROXY):

• every query the new pages issue returns 200 — remote and repository lists with ordering / name__icontains / pulp_label_select / remote filters, …/versions/, …/versions/?number=N, and distributions/deb/apt/?repository=<href>
• the exact create payload DebRemoteEdit sends → 201; the smartUpdate PUT with all APT fields → 202, and the fields read back verbatim
• the repository create plus auto-created distribution path → 201 / 202, and pluginRepositoryBasePath('deb', …) resolves the base_path the Repository URL field shows
• test objects cleaned up afterwards, existing content untouched

npm run lint:js, lint:ts, lint:ls, the this-without-class check and npm run build are all clean (build warnings unchanged at 18, all pre-existing bundle-size ones). No CHANGES.md entry, since that's generated from PR titles at release.

Two things I'd flag for review

1. A third copy of SyncModal. ansible-repository-sync.tsx and file-repository-sync.tsx already carry identical copies and I've followed suit rather than refactoring inside a feature PR. Happy to extract all three into a shared component, here or separately, if you'd prefer.
2. mirror defaults to true in the sync modal, matching the other plugins, though pulp_deb's own API default is false. I kept UI consistency, but say the word and I'll flip it — mirroring is the destructive direction.

@bmbouter @himdel @dkliban @ipanova

[Redtigercod4]

Thanks for this - solid work overall, and I've left comments on the specific issues, tagged using MoSCoW. A couple of things I'd suggest:

• Commit size: This is a large PR, 1,942 lines across 32 files submitted as a single commit. That makes it much harder for reviewers to follow, especially when trying to understand how the implementation came together. Smaller, sequential PRs would be preferred going forward.

• Cypress Testing: As this is new UI functionality within PulpUI, could you please add corresponding test cases to cypress/e2e/smoke.js.

• AI-assisted contributions: No issue either way, but if any AI tooling was used, please make sure the appropriate attributions is included, in line with Pulp's AI policy.

Ref:
https://github.com/pulp/.github/blob/cf562eff2ec6fe0f49bdfd693c3627453eef5203/profile/README.md

[warisshaikh1]

Thanks for the review -- all nine comments are addressed, force-pushed and rebased on current main. Details are in the individual threads; the three broader points:

Cypress. Added Deb repositories and Deb remotes to cypress/e2e/smoke.js. Both reach the list and its empty state, and the remotes case clicks through to the create form and checks the APT-only fields render, since those are what the shared RemoteForm gained here. Green in CI.

Commit size. Point taken. I have not retro-split the original commit, since the pages only work once the API layer, routes and menu entry are all present, but the review fixes are eight separate commits, one per point, so the diffs should be easy to follow. Anything further I send will be scoped smaller from the start -- starting with the SyncModal extraction, as its own PR.

AI-assisted. Yes, AI-assisted throughout, and thanks for pointing at the policy. Every commit now carries an Assisted By: Cursor (Claude Opus 5) trailer, including the original, which I amended. Everything here has been reviewed by me, and the deb pages are running against a live Pulp with a mirrored Ubuntu archive behind them, which is where the required-field and duplicate-path behaviours in the original commit message came from.

[ggainey]

@Redtigercod4 are you happy that this meets your list of suggested changes?

[Redtigercod4]

Thanks for the work on adding deb support and addressing the changes. I will speak to the core team regarding releasing this in the near future.

[warisshaikh1]

Done -- #363 merged yesterday, so this branch is rebased on main and now uses the shared SyncModal. Its own copy is gone and the deb sync action drops from 154 lines to 64.

One addition to the shared component: deb wants the mirror switch starting off, since that is what pulp_deb defaults a sync to, so SyncModal takes a defaultMirror prop. It defaults to what ansible and file have always sent, so their behaviour is unchanged and neither call site needed touching.

It is a boolean rather than a sync-params object on purpose. The effect that resets the switches has to depend on whatever carries that default, and an object prop would either lie about that dependency or, passed as a literal at the call site, reset the switches on every render. If you would rather it were an object, or want the mirror default handled some other way, happy to change it.

159ea94. Green in CI.

[Redtigercod4]

@ggainey Happy with the reviewed changes. Please feel free to merge.

[ggainey]

Good work, good discussion - thanks for the contribution and the responsiveness, both of you!