Objectcount

[GatewayJ]

Pull Request

Description

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update
• Code refactoring
• Performance improvement
• Test improvements
• Security fix

Testing

• Unit tests added/updated
• Manual testing completed

pnpm test:run

Checklist

• Code follows the project's style guidelines
• Self-review completed
• TypeScript types are properly defined
• All commit messages are in English (Conventional Commits)
• All existing tests pass
• No new dependencies added, or they are justified

Related Issues

Closes #

Screenshots (if applicable)

Additional Notes

[Copilot]

Pull request overview

This PR expands console permission evaluation and adjusts the browser buckets page so “Object Count” / “Size” columns can be shown even when the user can’t access the underlying usage endpoint (handling 403s locally instead of via the global redirect).

Changes:

• Extend ConsoleStatement parsing/evaluation to support NotAction and relax Action to optional, plus add console-scope resource handling logic.
• Add a per-request API client option to suppress the global 403 handler and throw instead.
• Update the buckets browser page to always request bucket usage and render placeholders ("--") when usage data can’t be fetched.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File	Description
lib/console-policy-parser.ts	Adds NotAction support and modifies action/resource matching for console scopes.
lib/api-client.ts	Introduces suppress403Redirect to bypass the global 403 handler and throw an error.
hooks/use-system.ts	Uses suppress403Redirect for /datausageinfo requests.
app/(dashboard)/browser/page.tsx	Always shows Object Count/Size columns and avoids populating 0-values when usage can’t be fetched.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

After removing isAdmin from the hook dependency arrays, isAdmin (and useAuth) appears to be unused in this component, which will trigger no-unused-vars during lint/typecheck. Consider removing the useAuth import and const { isAdmin } = useAuth() if it’s no longer needed.

[overtrue]

@copilot open a new pull request to apply changes based on this feedback

[overtrue]

@copilot fix conflicts

[overtrue]

@GatewayJ 处理一下冲突

[GatewayJ]

@GatewayJ 处理一下冲突

ok了 ,done it