Minor redundancy: Line 44 checks if strings.TrimSpace(stackID) is empty, and line 47 calls strings.TrimSpace(stackID) again when appending. The splitCommaList function already trims whitespace from each element, so the TrimSpace calls are redundant. Consider simplifying to:

for _, stackID := range splitCommaList(c.stacks) {
    if stackID == "" {
        continue
    }
    stacks = append(stacks, &tfe.Stack{ID: stackID})
}

The same pattern exists in variableset_update_workspaces.go lines 44-47.

Minor redundancy: Lines 54, 70, and 86 each call strings.TrimSpace on IDs that were already trimmed by splitCommaList. The splitCommaList function already trims whitespace from each element, so these TrimSpace calls are redundant. Consider simplifying to use the IDs directly from splitCommaList.

Inconsistent boolean flag parsing: Lines 67-74 manually parse the -global flag with if/else, but lines 76-91 use the parseBoolFlag helper function for -overridable and -agent-enabled. For consistency and maintainability, the -global flag should also use parseBoolFlag, similar to the other boolean flags.

The variableset read command is missing the include flag and related logic to pass include options to the API. The test file variableset_read_comprehensive_test.go includes a test TestVariableSetReadPassesIncludeOptions that expects this functionality, but the command implementation doesn't have:

1. An include field in the struct
2. The -include flag definition in FlagSet
3. Logic to parse and pass include options to the ReadAPI call

The command should be updated to support include options similar to how policyset_read.go handles them (lines 15, 24, 48-68).

The test TestVariableSetReadPassesIncludeOptions was added but the corresponding implementation for the -include flag is missing from the variableset read command. The test will fail because:

1. The command struct doesn't have an include field
2. The FlagSet doesn't define the -include flag
3. The Run method doesn't parse or use include options

Either the test should be removed (if include support is not intended for this PR), or the implementation needs to be added to variableset_read.go to support the -include flag similar to how policyset_read.go implements it.

The test TestVariableSetListPassesQueryAndInclude was added but the corresponding implementation for the -query and -include flags is missing from the variableset list command. The test will fail because:

1. The command struct doesn't have query and include fields
2. The FlagSet doesn't define the -query and -include flags
3. The Run method doesn't populate these options in VariableSetListOptions

Either the test should be removed (if query/include support is not intended for this PR), or the implementation needs to be added to variableset_list.go to support these flags similar to how policyset_list.go implements them.

Inconsistent options parameter passing: Line 75 passes options by value to RemoveFromProjects, while lines 59 and 91 pass &options by pointer to RemoveFromWorkspaces and RemoveFromStacks respectively. This should be consistent across all three calls unless the go-tfe API methods have different signatures. Verify that the go-tfe API for RemoveFromProjects expects a value rather than a pointer, or update to pass a pointer for consistency with the other methods.

Minor redundancy: Line 44 checks if strings.TrimSpace(workspaceID) is empty, and line 47 calls strings.TrimSpace(workspaceID) again when appending. The splitCommaList function already trims whitespace from each element, so the TrimSpace calls are redundant. Consider simplifying to:

for _, workspaceID := range splitCommaList(c.workspaces) {
    if workspaceID == "" {
        continue
    }
    workspaces = append(workspaces, &tfe.Workspace{ID: workspaceID})
}

Inconsistent options parameter passing: Line 84 passes options by value to ApplyToProjects, while lines 63 and 105 pass &options by pointer to ApplyToWorkspaces and ApplyToStacks respectively. This should be consistent across all three calls unless the go-tfe API methods have different signatures. Verify that the go-tfe API for ApplyToProjects expects a value rather than a pointer, or update to pass a pointer for consistency with the other methods.