Standardize agent output summary title to "Agentic Conversation"#15072
Standardize agent output summary title to "Agentic Conversation"#15072
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot make sure the special path for copilot to use the conversation.md file is also wrapped in a details view. |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in commit 3dc0490. The special Copilot path for |
There was a problem hiding this comment.
Pull request overview
Standardizes the GitHub Actions step summary section title for agent conversation output to a single, engine-agnostic label (“Agentic Conversation”), and ensures Copilot’s conversation.md fast-path output is wrapped consistently with other parsing flows.
Changes:
- Updated
wrapAgentLogInSection()to always emit<summary>Agentic Conversation</summary>(no parser-specific title). - Wrapped Copilot
conversation.md(--share) step summary output in the same details/summary section. - Updated affected unit tests to match the new static title and wrapping behavior.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| actions/setup/js/log_parser_shared.cjs | Hard-codes the agent summary title to “Agentic Conversation”. |
| actions/setup/js/log_parser_bootstrap.cjs | Wraps Copilot conversation.md output using wrapAgentLogInSection(). |
| actions/setup/js/log_parser_shared.test.cjs | Updates expectations for the new static summary title. |
| actions/setup/js/log_parser_bootstrap.test.cjs | Updates expectations to validate wrapped output and title in step summaries. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const openAttr = open ? " open" : ""; | ||
| const emoji = "🤖"; | ||
| const title = `${emoji} ${parserName} CLI Session`; | ||
| const title = "Agentic Conversation"; | ||
|
|
||
| return `<details${openAttr}>\n<summary>${title}</summary>\n\n${markdown}\n</details>`; |
There was a problem hiding this comment.
wrapAgentLogInSection now hard-codes the summary title, so parserName (and its JSDoc describing it) no longer affects output. To avoid confusion for future maintainers, please update the function contract to reflect this (e.g., note parserName is accepted only for backward compatibility and ignored, or rename the destructured variable to _parserName / remove it from destructuring while still accepting it in options).
| const markdown = "Test content"; | ||
| const result = wrapAgentLogInSection(markdown, { parserName: "Claude" }); | ||
|
|
||
| expect(result).toContain("🤖 Claude CLI Session"); | ||
| expect(result).toContain("Agentic Conversation"); |
There was a problem hiding this comment.
This test name/intent is now inconsistent with the behavior being asserted: it says "should support custom parser names", but the title is intentionally static. Rename the test (and optionally stop passing parserName here) to reflect that parserName is ignored for the summary title and is retained only for backward compatibility.
Agent conversation output in GitHub Actions step summaries currently displays parser-specific titles (
🤖 Copilot CLI Session,🤖 Claude CLI Session, etc.). This changes the HTML details element to use a consistent title across all AI engines.Changes
actions/setup/js/log_parser_shared.cjs: ModifiedwrapAgentLogInSection()to return static title"Agentic Conversation"instead of${emoji} ${parserName} CLI Sessionactions/setup/js/log_parser_bootstrap.cjs: Added wrapping for Copilot's specialconversation.mdpath (generated by--shareflag) to ensure consistent output across all code pathslog_parser_shared.test.cjsandlog_parser_bootstrap.test.cjsBefore:
After:
The
parserNameparameter remains accepted for backward compatibility but no longer affects the output title. Both regular log file parsing and Copilot'sconversation.mdspecial path now produce consistent wrapped output.💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.