Skip to content

Add missing pipeline steps to frontend status indicator #162

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
CyberSecDef merged 2 commits into from
Apr 9, 2026
+12 −0
Merged

Add missing pipeline steps to frontend status indicator #162

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3e24036
Initial plan
Copilot Apr 9, 2026
3555e39
Add missing pipeline steps to inferStatusFromRequestEntry
Copilot Apr 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions static/js/script.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -260,6 +260,18 @@ $(function () {
if (combined.indexOf("revise") !== -1 || combined.indexOf("revision") !== -1) {
return "Applying Chapter Revisions";
}
if (combined.indexOf("voice & dialogue") !== -1 || combined.indexOf("voice and dialogue") !== -1) {
return "Differentiating character voices";
}
Comment on lines +263 to +265
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This condition is effectively unreachable/inaccurate because an earlier check matches any combined containing "dialogue" and returns "Refining Chapter Dialog" first. Also, the rendered prompt text uses the phrase "dialogue and voice" (not "voice & dialogue" / "voice and dialogue"), so this may never match when inferring from payload.messages. Consider moving a more specific voice+dialogue detection above the generic "dialogue" check and matching the actual prompt wording (or using a bounded regex like voice.*dialogue / dialogue.*voice).

Copilot uses AI. Check for mistakes.
if (combined.indexOf("human oddities") !== -1) {
return "Adding human texture";
}
Comment on lines +266 to +268
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The human-oddities prompt text uses the singular phrase "human oddity" (see prompts.yml), so searching for "human oddities" in payload.messages won’t match and will still fall back to the generic status. Adjust the substring (e.g., match "human oddity") or include entry.action in the string you scan.

Copilot uses AI. Check for mistakes.
if (combined.indexOf("metaphor reduction") !== -1) {
return "Reducing metaphor density";
}
Comment on lines +269 to +271
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The metaphor-reduction prompt content does not contain the literal phrase "metaphor reduction"; it uses wording like "reduce the metaphor density" / "over-metaphorise". If you’re inferring from payload.messages, this check likely won’t match and will keep showing the fallback status. Consider matching the actual prompt phrasing or incorporating entry.action into the scanned text.

Copilot uses AI. Check for mistakes.
if (combined.indexOf("copy edit") !== -1) {
return "Copy editing";
}

return "Prompting LLM";
}
Expand Down
Loading