mirror of
https://github.com/usebruno/bruno.git
synced 2026-07-08 06:28:33 +00:00
646c90819d
* feat: enhance ScriptError with source context, code snippets, and navigation Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat: remove auto-commenting of untranslated pm commands during import Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat: update CodeSnippet styles to use theme colors for error and warning highlights * fix: remove unused SCRIPT_TYPES import from network IPC module * refactor: remove unused functions and clean up source-context utility - Removed `getUnifiedScriptContext`, `getWarningSourceGroups`, and related helper functions from `source-context.js` to streamline the utility. - Updated tests in `source-context.spec.js` to reflect the removal of unused functions, ensuring only relevant tests for `findLineInSource` and `getScriptContext` remain. * refactor: simplify ScriptError component and update styles * refactor: streamline tab management in Script components * refactor: enhance tab management in ScriptError and add testsMetadata handling in prepare-request * refactor: improve error source identification in ScriptError component - Enhanced the logic for determining error source types by introducing separate checks for folder and collection files. - Updated the handling of folder file names to ensure accurate UID retrieval and labeling. - Streamlined the overall structure of the getErrorSourceInfo function for better readability and maintainability. * refactor: improve ScriptError component and enhance styling - Simplified the conditions for displaying the ScriptError component in ResponsePane. - Updated navigation logic in ScriptErrorCard to ensure proper handling of source information. - Adjusted styles in StyledWrapper to allow for visible overflow. - Enhanced ScriptErrorIcon to accept additional class names for better styling flexibility. - Minor layout adjustments in RunnerResults ResponsePane for improved UI consistency. * refactor: simplify test description for untranslated pm commands and consolidate error formatter imports * fixes * refactor: update focusedTab logic in Script components to use collection and folder UIDthe respective collection and folder UID instead of the activeTabUid. * feat: add buildErrorContext utility for enhanced error handling in network IPC - Introduced a new utility function `buildErrorContext` to construct detailed error context from script errors. - This function parses error locations, adjusts line numbers, and retrieves relevant source context, improving error reporting. - Removed the previous inline error handling logic from the network IPC module to streamline the codebase. * feat: added playwright test cases to test scriptError behavior * refactor: enhance ScriptError component and improve error handling - Updated labels for error source types in ScriptError to be more concise (e.g., "Request Script" to "Request"). - Improved navigation logic in ScriptErrorCard to ensure proper handling of navigable file paths. - Enhanced styling in StyledWrapper for better visual consistency and user experience. - Added tests to verify fallback behavior for missing error types and non-navigable file paths. - Refined utility functions for better context extraction from script errors. * refactor: normalize file paths for cross-platform compatibility in ScriptError component * refactor: improve file path handling in ScriptError component * refactor: enhance buildErrorContext for improved error handling * docs: add detailed comments to build-error-context for better understanding of error handling * refactor: enhance error block line detection in error formatter - Updated `findScriptBlockEndLine` and `findYmlScriptBlockEndLine` functions to return null for empty or missing blocks, improving accuracy in line detection. - Added comprehensive tests for both functions to ensure correct behavior across various scenarios, including handling of non-.bru and non-.yml files. * refactor: improve error handling and testing in ScriptError and buildErrorContext - Updated ScriptError component to streamline tab management by replacing focusTab with addTab for better request handling. - Enhanced buildErrorContext to return null for empty or missing script blocks in .bru and .yml files, ensuring accurate error reporting. - Added tests to validate behavior for empty script blocks and improved error context extraction in various scenarios. * test: add new tests for script error navigation and handling - Implemented tests for post-response file-path navigation to the Script tab and verification of active sub-tabs. - Added keyboard navigation tests to trigger file-path navigation using the Enter key. - Included a test for multiple error cards to ensure closing one does not affect others. - Enhanced runner tests to verify navigation to the Tests tab from script error results. * refactor: enhance CodeSnippet line rendering and remove unused source-context utilities * refactor: update locators in script-errors tests for improved readability and maintainability * test: enhance script-errors tests to verify error line content * review fixes * refactor: update RunnerResults component and enhance locators for improved testability * refactor: remove buildErrorContext and replace with formatErrorWithContextV2 - Deleted the buildErrorContext function and its associated tests. - Updated network IPC to utilize formatErrorWithContextV2 for improved error context handling. - Enhanced error reporting by ensuring structured error context is returned for desktop UI. * refactor: enhance tab components with data-testid attributes for improved testability - Added data-testid attributes to tab elements in CollectionSettings and FolderSettings components for better integration with testing frameworks. - Updated Tabs and ResponsiveTabs components to include data-testid attributes for tab triggers, enhancing the ability to select and verify tabs in tests. - Modified script-errors tests to utilize new locators for improved readability and maintainability. --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
62 lines
1.5 KiB
JavaScript
62 lines
1.5 KiB
JavaScript
import styled from 'styled-components';
|
|
import { rgba } from 'polished';
|
|
|
|
const StyledWrapper = styled.div`
|
|
.code-snippet {
|
|
font-family: monospace;
|
|
font-size: ${(props) => props.theme.font.size.xs};
|
|
line-height: 1.4;
|
|
overflow-x: auto;
|
|
border-radius: ${(props) => props.theme.border.radius.base};
|
|
background-color: ${(props) => props.theme.background.elevated};
|
|
border: 1px solid ${(props) => props.theme.border.border2};
|
|
}
|
|
|
|
.code-line {
|
|
display: flex;
|
|
align-items: stretch;
|
|
}
|
|
|
|
.code-line.highlighted-error {
|
|
background-color: ${(props) => rgba(props.theme.colors.text.danger, 0.1)};
|
|
border-left: 3px solid ${(props) => props.theme.colors.text.danger};
|
|
}
|
|
|
|
.code-line.highlighted-warning {
|
|
background-color: ${(props) => rgba(props.theme.colors.text.warning, 0.1)};
|
|
border-left: 3px solid ${(props) => props.theme.colors.text.warning};
|
|
}
|
|
|
|
.code-line:not(.highlighted-error):not(.highlighted-warning) {
|
|
border-left: 3px solid transparent;
|
|
}
|
|
|
|
.code-line-number {
|
|
min-width: 2.5rem;
|
|
text-align: right;
|
|
padding: 0 0.5rem;
|
|
color: ${(props) => props.theme.colors.text.muted};
|
|
user-select: none;
|
|
flex-shrink: 0;
|
|
}
|
|
|
|
.code-line-content {
|
|
white-space: pre;
|
|
padding: 0 0.5rem;
|
|
flex: 1;
|
|
min-width: 0;
|
|
}
|
|
|
|
.code-line-separator {
|
|
border-left: 3px solid transparent;
|
|
}
|
|
|
|
.separator-content {
|
|
color: ${(props) => props.theme.colors.text.muted};
|
|
user-select: none;
|
|
padding: 0 0.5rem;
|
|
}
|
|
`;
|
|
|
|
export default StyledWrapper;
|