Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

quality: SC2030 PATH modification is local to subshell in tests/test-ai-supervisor-e2e.sh #3048

@marcusquinn

Description

@marcusquinn

Found by daily quality sweep on 2026-03-07.

Source: ShellCheck (SC2030)
Severity: medium
Files affected: tests/test-ai-supervisor-e2e.sh

Finding: Modification of PATH is local (to subshell caused by (..) group).

The PATH modification inside a (..) subshell group does not affect the parent shell. This is likely a logic bug — the test intends to modify PATH for subsequent commands but the modification is silently discarded.

Recommended fix: Move the PATH modification outside the subshell group, or use { ... } (brace group) instead of ( ... ) (subshell group) if the intent is to keep changes in the current shell.

Ref: quality review issue #2632

Metadata

Metadata

Assignees

Labels

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions