diff --git a/src/components/AccessScopes/Forms/AccessScopeCriteriaForm.tsx b/src/components/AccessScopes/Forms/AccessScopeCriteriaForm.tsx index 228c53bbd..594b4e861 100644 --- a/src/components/AccessScopes/Forms/AccessScopeCriteriaForm.tsx +++ b/src/components/AccessScopes/Forms/AccessScopeCriteriaForm.tsx @@ -83,7 +83,7 @@ export default function AccessScopeCriteriaForm({ className="rounded border border-gray-300 bg-gray-50 p-3" >
- {/* Display per-scope validation error (except wildcard errors, which are shown inline) */} + {/* Display per-scope validation error */} {(() => { const error = showError && @@ -91,8 +91,7 @@ export default function AccessScopeCriteriaForm({ scopesError[index]; return ( error && - typeof error === "string" && - !error.includes("Wildcard") && ( + typeof error === "string" && (

{error}

) ); @@ -130,13 +129,7 @@ export default function AccessScopeCriteriaForm({
- + {scope.names && scope.names.trim().length > 0 && (() => { @@ -144,33 +137,28 @@ export default function AccessScopeCriteriaForm({ .split("\n") .map((n: string) => n.trim()) .filter(Boolean); - const hasWildcard = names.includes("*"); - const hasWildcardError = hasWildcard && names.length > 1; return ( - <> -
-
- {names.map((name: string, nameIndex: number) => ( - - ))} -
+
+
+ {names.map((name: string, nameIndex: number) => ( + + ))}
- {hasWildcardError && ( -

- Wildcard '*' must be the only name when used -

- )} - +
); })()} +
diff --git a/src/components/AccessScopes/Forms/AccessScopeForm.tsx b/src/components/AccessScopes/Forms/AccessScopeForm.tsx index ceb672698..eefc1ae67 100644 --- a/src/components/AccessScopes/Forms/AccessScopeForm.tsx +++ b/src/components/AccessScopes/Forms/AccessScopeForm.tsx @@ -164,21 +164,6 @@ export default function AccessScopeForm({ errors.scopes[index] = "Each scope must have at least one of: tags, agents, or names"; } - - // Validate wildcard usage in names - if (scope.names && scope.names.trim().length > 0) { - const names = scope.names - .split("\n") - .map((n: string) => n.trim()) - .filter(Boolean); - - const hasWildcard = names.includes("*"); - if (hasWildcard && names.length > 1) { - if (!errors.scopes) errors.scopes = []; - errors.scopes[index] = - "Wildcard '*' must be the only name when used"; - } - } }); }