From b6293434e99cccdf6abe320cd26000f094bca1e5 Mon Sep 17 00:00:00 2001
From: xiezhouwei <xzw@qx002575.com>
Date: Tue, 23 Jun 2026 14:06:26 +0800
Subject: [PATCH] feat: hide channel, user, and retry columns in usage log
 table by default\n\n- Set CHANNEL/USERNAME/RETRY default visibility to false
 for all users\n- Remove admin-only override logic (no longer differentiate by
 role)\n- Exclude these 3 columns from 'Select All' operation

---
 web/src/hooks/usage-logs/useUsageLogsData.jsx | 26 +++++++------------
 1 file changed, 10 insertions(+), 16 deletions(-)

diff --git a/web/src/hooks/usage-logs/useUsageLogsData.jsx b/web/src/hooks/usage-logs/useUsageLogsData.jsx
index 90b8058..fab58b7 100644
--- a/web/src/hooks/usage-logs/useUsageLogsData.jsx
+++ b/web/src/hooks/usage-logs/useUsageLogsData.jsx
@@ -476,8 +476,8 @@ export const useLogsData = () => {
   const getDefaultColumnVisibility = () => {
     return {
       [COLUMN_KEYS.TIME]: true,
-      [COLUMN_KEYS.CHANNEL]: isAdminUser,
-      [COLUMN_KEYS.USERNAME]: isAdminUser,
+      [COLUMN_KEYS.CHANNEL]: false,
+      [COLUMN_KEYS.USERNAME]: false,
       [COLUMN_KEYS.TOKEN]: true,
       [COLUMN_KEYS.GROUP]: true,
       [COLUMN_KEYS.TYPE]: true,
@@ -486,7 +486,7 @@ export const useLogsData = () => {
       [COLUMN_KEYS.PROMPT]: true,
       [COLUMN_KEYS.COMPLETION]: true,
       [COLUMN_KEYS.COST]: true,
-      [COLUMN_KEYS.RETRY]: isAdminUser,
+      [COLUMN_KEYS.RETRY]: false,
       [COLUMN_KEYS.IP]: true,
       [COLUMN_KEYS.DETAILS]: true,
     };
@@ -503,13 +503,6 @@ export const useLogsData = () => {
     try {
       const parsed = JSON.parse(savedColumns);
       const merged = { ...defaults, ...parsed };
-
-      if (!isAdminUser) {
-        merged[COLUMN_KEYS.CHANNEL] = false;
-        merged[COLUMN_KEYS.USERNAME] = false;
-        merged[COLUMN_KEYS.RETRY] = false;
-      }
-
       return merged;
     } catch (e) {
       console.error('Failed to parse saved column preferences', e);
@@ -576,13 +569,14 @@ export const useLogsData = () => {
     const allKeys = Object.keys(COLUMN_KEYS).map((key) => COLUMN_KEYS[key]);
     const updatedColumns = {};
 
+    const hiddenColumns = [
+      COLUMN_KEYS.CHANNEL,
+      COLUMN_KEYS.USERNAME,
+      COLUMN_KEYS.RETRY,
+    ];
+
     allKeys.forEach((key) => {
-      if (
-        (key === COLUMN_KEYS.CHANNEL ||
-          key === COLUMN_KEYS.USERNAME ||
-          key === COLUMN_KEYS.RETRY) &&
-        !isAdminUser
-      ) {
+      if (hiddenColumns.includes(key)) {
         updatedColumns[key] = false;
       } else {
         updatedColumns[key] = checked;