Repository navigation
FEAT: Adding Temporary Instance Creation from the Registry - #3030
Merged
Richard Lundeen (richlundeen) merged 23 commits intoOct 20, 2026
Merged
Richard Lundeen (richlundeen) merged 23 commits into
Richard Lundeen (richlundeen) merged 23 commits into
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Correct declared constructor capture, clear temporary overrides, and guard runtime readiness during target preparation. Refine converter controls and move chat settings into a separate ribbon. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Constrain target mocks to PromptTarget so optional source attributes are not created implicitly. Cover shared-source request pacing and lock replacement across event loops. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Keep temporary component lifecycle and upstream dynamic target parameter metadata. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: d9a40875-faa1-48ad-bf6b-8d49a4ad7eb8
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: d9a40875-faa1-48ad-bf6b-8d49a4ad7eb8
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…ary-component-settings' into richlundeen-temporary-component-settings
Keep external-input validation and typed converter forms. Give repeated sends independent temporary targets and require applied previews for private converter settings. Remove the narrow-screen layout note from the GUI guide. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Keep temporary target and converter types alongside the technique registry types. Use the unit test package import to avoid optional dependency package shadowing. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8db9da14-941e-4b51-bbfe-556c43058dbe
This was referenced Oct 13, 2026
Retain temporary target construction and cleanup alongside CoPyRIT GitHub token validation. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 15d4b93d-85a2-45d8-9f11-aed28ebfa41c
Preserve upstream request limits, server upload controls, and administrator checks alongside optional registration and source reconstruction. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
|
I see the workflow for unregistered targets--validate the variant, derive its identifier and capabilities, save a source-plus-overrides recipe, and reconstruct it for sends. but not sure i understand the usecase for unregistered scorers.i might be missing something butcurrently, |
Keep target construction on the request event loop. Preserve save errors and roll back media when private target cleanup fails. Retain resolved source authentication and integrate converter run tracking. Clarify temporary target cleanup and scorer preview limits. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
Author
|
hannahwestra25 There are use cases for temporary scorers that are not implemented yet. As we add rescoring of individual pieces in the GUI, it will be useful to change scorer settings without changing the registered instance. I'd rather keep scorers in the same construction pattern than exclude them because those workflows are not in place yet; the support costs almost nothing. I removed the scorer-specific documentation. Written with GitHub Copilot. |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Retain Operation attribution and temporary-target bindings during attack creation. Cover repeated sends with both Operation IDs and legacy Operation names. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
hannahwestra25
approved these changes
Oct 19, 2026
Use registry external-input validation for unregistered scorers. Restore converter pipelines through the shared stage comparison so changed or cleared temporary settings are applied. Preserve main's media checks and credential permissions alongside private component construction. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
… workflow Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Richard Lundeen (richlundeen)
enabled auto-merge
October 19, 2026 23:06
Keep the HuggingFace request-loop regression independent of optional Torch and remove a redundant cleanup guard rejected by unit typing. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Include the upstream Copilot model-identity test isolation fix without changing temporary component behavior. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
github-merge-queue
Bot
removed this pull request from the merge queue due to a conflict with the base branch
Oct 20, 2026
Preserve temporary converter settings and source targets while retaining adversarial defaults for newly registered converters. Add an integration regression for the combined behavior. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Keep the branch current after the converter-conflict resolution, including main's Operations E2E stabilization. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
The default-label loading notice can disappear between the badge and timestamp measurements. Wait for the existing app readiness condition before taking geometry measurements; keep all overlap and overflow assertions unchanged. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Richard Lundeen (richlundeen)
deleted the
richlundeen-temporary-component-settings
branch
October 20, 2026 02:54
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds support for creating targets, converters, and scorers without adding them to a registry.
For example, say a user wants to change settings for one conversation or conversion without changing shared registry entries. Temporary instances support those changes without adding a registry entry for every variant.
I added two changes to the GUI of when this is useful already (but have some bigger arcs in mind)
Example 1: When starting a new attack, you can modify the temperature without modify the registered target
Example 2: You can modify an existing converter without adding it into the registry. E.g. change a language from Spanish to French without adding a whole other converter
The backend adds support by using the existing "create" endpoints accept
register: falseand use registry construction without registration. Source-based overrides, saved target recipes, and scoped converter previews reuse this path.