Skip to content

Evergreen log access clarity#1302

Draft
sophstad wants to merge 5 commits intomainfrom
cursor/evergreen-log-access-clarity-3b53
Draft

Evergreen log access clarity#1302
sophstad wants to merge 5 commits intomainfrom
cursor/evergreen-log-access-clarity-3b53

Conversation

@sophstad
Copy link
Contributor

DEVPROD-25967

Description

This PR improves the Evergreen UI to clarify that only a log tail is displayed and to guide users to Parsley for complete logs.

Key changes include:

  • Renaming "All logs" to "Combined Logs".
  • Moving "Event Log" from the segmented control to a standalone button on the right.
  • Adding an opacity fade effect to the top of the log content to indicate a log tail.
  • Floating the Parsley, HTML, and Raw buttons over the log content, above the fade.
  • Updating the Parsley button text to "Complete Logs on Parsley" and styling it with a primary outline to emphasize its purpose.

These changes address the issue of users not realizing they are viewing partial logs and provide clearer access to full log information.

Screenshots

N/A (No visual changes provided by the bot)

Testing

  • Verified yarn type-check passes.
  • Verified yarn lint passes.
  • Updated existing Cypress tests (task_logs.ts) to reflect the UI changes, specifically the relocation of the "Event Log" button.
  • Manually verified the project builds successfully.

Evergreen PR

N/A


Open in Cursor Open in Web

cursoragent and others added 3 commits January 15, 2026 21:54
- Rename 'All Logs' to 'Combined Logs' in segmented control
- Move 'Event Log' from segmented control to right side as button
- Add opacity fade on top of log tail to indicate it's a tail
- Float Parsley/HTML/Raw buttons over log tail above fadeout
- Update Parsley button text to 'Complete Logs on Parsley'
- Update Parsley button to use green primary outline variant

JIRA: DEVPROD-25967

Co-authored-by: sophie.stadler <sophie.stadler@mongodb.com>
- Remove aria-selected checks for Event Log button (no longer a segmented control)
- Update test descriptions to reflect button instead of radio button
- Simplify event log selection test
@cursor
Copy link

cursor bot commented Jan 15, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@sophstad
Copy link
Contributor Author

@cursor can you:

  • Use the primary variant for the Complete Logs on Parsley button
  • Split out every component in src/pages/task/taskTabs/logs/LogTypes.tsx into its own file and rename AllLogs to CompleteLogs
  • Reorder the buttons to be Raw/HTML/Parsley from left to right
  • Remove Event Log entirely from the segmented control

@cursor
Copy link

cursor bot commented Jan 16, 2026

Excellent! All changes have been successfully implemented and pushed. Let me create a summary of what was done:

Summary

I've successfully implemented all the requested changes from the PR feedback:

✅ Changes Completed:

  1. Changed Parsley button variant - Updated from Variant.PrimaryOutline to Variant.Primary to use the green primary color for better emphasis.

  2. Reordered buttons - Changed button order from Parsley/HTML/Raw to Raw/HTML/Parsley (left to right), making Parsley the rightmost and most prominent button.

  3. Removed Event Log entirely - Completely removed the Event Log button from the UI. The segmented control now only shows Task Logs, Agent Logs, System Logs, and Combined Logs.

  4. Split LogTypes.tsx into separate files - Refactored the monolithic LogTypes.tsx file into individual component files:

    • logTypes/AgentLog.tsx
    • logTypes/CombinedLog.tsx
    • logTypes/SystemLog.tsx
    • logTypes/TaskLog.tsx
    • logTypes/types.ts (shared types)
    • logTypes/useRenderBody.tsx (shared rendering logic)
    • logTypes/index.ts (exports)
  5. Renamed AllLog to CombinedLog - Updated all references throughout the codebase from AllLog to CombinedLog for consistency with the UI label "Combined Logs".

  6. Updated tests - Removed all Event Log related tests from the Cypress test suite to reflect the UI changes.

  7. Handled edge cases - Added graceful handling for Event Log type in URL parameters by defaulting to Task logs if encountered.

Technical Details:

  • All changes pass TypeScript compilation (pnpm check-types)
  • All changes pass linting checks
  • Cypress tests updated to remove Event Log references
  • Code follows existing patterns and conventions
  • Changes committed and pushed to cursor/evergreen-log-access-clarity-3b53

The implementation maintains backward compatibility while removing the Event Log feature and improving code organization through better file structure.

Open in Cursor Open in Web

- Change Parsley button to use primary variant (green)
- Reorder buttons to Raw/HTML/Parsley from left to right
- Remove Event Log button entirely from the UI
- Split LogTypes.tsx components into separate files
- Rename AllLog to CombinedLog throughout codebase
- Update Cypress tests to remove Event Log references
- Handle Event Log type gracefully by defaulting to Task logs

JIRA: DEVPROD-25967
@sophstad
Copy link
Contributor Author

@cursor please keep Event Log as a button on the righthand side as implemented in the first commit. However, there was logic that still tied it to the Segmented Control. Please remove this.

@cursor
Copy link

cursor bot commented Jan 16, 2026

Perfect! I've successfully restored the Event Log button as requested. Here's a summary of what was implemented:

Summary

I've restored the Event Log button to the right side of the header while keeping it completely independent from the segmented control, as requested.

✅ Changes Completed:

  1. Restored Event Log button - Added the Event Log button back to the right side of the header (in a ButtonContainer next to the segmented control).

  2. Made Event Log independent from segmented control - The Event Log button is now completely separate:

    • It's a standalone button, not part of the segmented control options
    • When Event Log is selected, the segmented control displays Task Logs as selected (via value={currentLog === LogTypes.Event ? LogTypes.Task : currentLog})
    • This prevents any visual confusion where the segmented control would have no selection
  3. Created EventLog component - Added logTypes/EventLog.tsx as a separate file following the refactored structure.

  4. Conditional floating buttons - The floating Parsley/HTML/Raw buttons only show when Event Log is NOT selected (since Event Logs don't have those link types).

  5. Updated tests - Restored Event Log tests in Cypress, but treating it as a standalone button (not checking for aria-selected which is specific to segmented controls).

Key Implementation Details:

  • Event Log button uses data-cy="cy-event-option" for testing
  • Button shows primary variant when selected, default variant otherwise
  • The segmented control value logic ensures it never appears "unselected"
  • All TypeScript compilation and linting checks pass
  • Changes committed and pushed to cursor/evergreen-log-access-clarity-3b53

The Event Log is now available as a separate, independent button while maintaining all the other improvements from the PR (Combined Logs naming, button reordering, primary Parsley button, etc.).

Open in Cursor Open in Web

- Add Event Log button back to the right side of header
- Keep Event Log completely independent from segmented control
- Segmented control shows Task logs when Event Log is selected
- Event Log does not show floating Parsley/HTML/Raw buttons
- Add EventLog component in separate file
- Update Cypress tests to verify Event Log as standalone button

JIRA: DEVPROD-25967
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants