salvacybersec/feynman
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: tighten workflow prompts and search defaults

Browse Source
This commit is contained in:
Advait Paliwal
2026-04-14 09:30:15 -07:00
parent af6486312d
commit 0995f5cc22
11 changed files with 26 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
tests/pi-web-access-patch.test.ts
View File

@@ -33,13 +33,15 @@ test("patchPiWebAccessSource updates index.ts directory handling", () => {
assert.match(patched, /const dir = dirname\(WEB_SEARCH_CONFIG_PATH\);/);
});
test("patchPiWebAccessSource defaults workflow to none for index.ts", () => {
test("patchPiWebAccessSource defaults workflow to none for index.ts without disabling explicit summary-review", () => {
const input = [
'function resolveWorkflow(input: unknown, hasUI: boolean): WebSearchWorkflow {',
'\tif (!hasUI) return "none";',
'\tif (typeof input === "string" && input.trim().toLowerCase() === "none") return "none";',
'\treturn "summary-review";',
'}',
'const configWorkflow = loadConfigForExtensionInit().workflow;',
'const workflow = resolveWorkflow(params.workflow ?? configWorkflow, ctx?.hasUI !== false);',
'workflow: Type.Optional(',
'\tStringEnum(["none", "summary-review"], {',
'\t\tdescription: "Search workflow mode: none = no curator, summary-review = open curator with auto summary draft (default)",',
@@ -50,8 +52,9 @@ test("patchPiWebAccessSource defaults workflow to none for index.ts", () => {
const patched = patchPiWebAccessSource("index.ts", input);
assert.match(patched, /return "none";/);
assert.doesNotMatch(patched, /summary-review = open curator with auto summary draft \(default\)/);
assert.match(patched, /params\.workflow \?\? configWorkflow \?\? "none"/);
assert.match(patched, /return "summary-review";/);
assert.match(patched, /summary-review = open curator with auto summary draft \(opt-in\)/);
});
test("patchPiWebAccessSource is idempotent", () => {