Description-only preset edits split comma-containing exact goals into separate goals
维护者通常 2 天内回复
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 78/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- react, typescript
- 领域
- frontend
调研方向
从 frontend/src/components/Scenarios/parameterForm.ts 开始，尤其关注第 103–107 行的列表序列化，并追踪 ScenarioPresetEditor 如何提交未更改的目标值。无损保留列表值，使得仅编辑描述时仍能保留目标内容、顺序、重复项和空列表；验证已报告的目标包含逗号的情况以及列出的其他边缘情况。
由索引模型根据 Issue 内容生成。
描述
Describe the bug
Editing only a scenario preset description serializes an unconstrained list of exact goals as comma-joined text, then splits commas inside individual goal strings into new goals.
Steps/Code to Reproduce
Open a preset with goals ['Hello, world!', 'Second goal'], edit only Description, and save through the ScenarioPresetEditor form.
Expected Results
Unrelated edits preserve the original list elements and order exactly: ['Hello, world!', 'Second goal'].
Actual Results
Editor submitted ['Hello', 'world!', 'Second goal']; goal count changed from two to three and behavioral preset identity changed. Both old and new payloads pass the actual scenario preset/configuration validators, so validation does not detect the data loss.
Verified source reference: parameter form list serialization.
Proposed fix/regression: keep list values structured or use a lossless representation; preserve untouched stored values. Cover commas, significant whitespace, empty lists, duplicates, and order. This differs from empty SearchReplace scalar strings (finding 48).
Screenshots
Not applicable; submitted payload is inline.
Versions
Tested commit d3446e045960b5ebee4641aa70fdeb03af186587, PyRIT 1.2.0.dev0, offline actual React component/router with deterministic service mocks; backend model/config validators accepted both payloads. Evidence: files/evidence/round12/comma-goal-payload.json and regressions.txt; local paths are not attachments.
- 主要语言
- Python
- 星标
- 4.7k
- 派生
- 1k
- 平均合并
- 2 天 2 小时
- 30 天内合并 PR
- 439
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
microsoft/PyRIT 的其他 Issue
-
GUI rejects valid empty-string SearchReplaceConverter settings, preventing deletion converters可能已有人在做 @Darshika140 于 1 天前认领。 未关闭help wanted
难度 2/5 1-3 小时 新手友好度 76/100
microsoft/PyRIT#3749 · 1 条评论 ·
维护者通常 2 天内回复
-
HTML scenario export replaces an intentionally empty converted message with the original text可能已有人在做 @VISHNU0906 于 1 天前认领。 未关闭help wanted
难度 2/5 1-3 小时 新手友好度 77/100
microsoft/PyRIT#3741 · 1 条评论 · 已指派 1 人 ·
维护者通常 2 天内回复
-
help wanted
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 2 天内回复
-
Development toolchain includes an audited vulnerable Zod version可能已有人在做 @ppcvote 于 1 天前认领。 未关闭help wanted
难度 2/5 1-3 小时 新手友好度 70/100
microsoft/PyRIT#3725 · 1 条评论 ·
维护者通常 2 天内回复
-
TextTarget is described as using API-key authentication even though it needs no credentials可能已有人在做 @pcbeingused333 于 1 天前认领。 未关闭help wanted
难度 2/5 1-3 小时 新手友好度 74/100
维护者通常 2 天内回复
相似的 Issue
-
question
难度 2/5 1-3 小时 新手友好度 62/100
conda-forge/vc-feedstock#132 ·
-
难度 2/5 1-3 小时 新手友好度 72/100
limitlessprojects/Go-Hybrid#3342 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 85/100
UKGovernmentBEIS/inspect_ai#6268 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 65/100
LSSTDESC/rail_sompz#42 ·
-
难度 2/5 1-3 小时 新手友好度 66/100
pytest-dev/pytest-forked#113 ·