Compare commits

...

4 Commits

Author SHA1 Message Date
Typer_Body 3d5e30c2e6 admin-check-group-chat 2026-06-22 01:29:32 +08:00
RockChinQ a2cdbb621b Merge branch 'fix/mobile-responsive-pages': mobile responsive fixes for dashboard, plugin detail & models dialog 2026-06-20 10:58:43 -04:00
RockChinQ b92d54254b fix(web): improve mobile responsiveness of dashboard, plugin detail & models dialog
- monitoring: stack filters full-width, scrollable tab bar, reduce card/content padding on mobile
- models dialog: provider form modal no longer overflows viewport on small screens; shared panel body padding shrinks on mobile
- plugin logs: reduce horizontal padding on mobile
2026-06-20 10:42:35 -04:00
huanghuoguoguo d22fa82d7c fix(skills): bootstrap generated lbs wrapper (#2270) 2026-06-20 17:24:04 +08:00
13 changed files with 175 additions and 73 deletions
+1
View File
@@ -39,6 +39,7 @@ bin/lbs index
```
Use `bin/lbs env show` to inspect defaults and `bin/lbs env doctor` when diagnosing local environment readiness. Env output is redacted by default; do not work around that by printing raw secrets.
`bin/lbs` is a generated local wrapper. If it is missing on a fresh checkout, run `npm run bootstrap` from this directory first; `npm install` also regenerates it via `prepare`.
Use `bin/lbs fixture check` before fixture-heavy cases such as MCP, RAG, multimodal, or plugin smoke tests.
Use `bin/lbs case list --ready` for cases that have no missing machine inputs and no manual preconditions. Use `bin/lbs case list --machine-ready` when you want to keep `manual-check` candidates and confirm their preconditions yourself.
+5 -1
View File
@@ -35,9 +35,13 @@ and LangBot's own Local Agent) working with the LangBot ecosystem.
## The `lbs` CLI
The testing assets ship with a small CLI (`bin/lbs`, Node 22.6):
The testing assets ship with a small CLI (`bin/lbs`, Node >= 22.6). The
`bin/lbs` wrapper is a generated local entrypoint; on a fresh checkout, run
`npm run bootstrap` once if it is missing. `npm install` also regenerates it via
the `prepare` script.
```bash
npm run bootstrap # create bin/lbs if missing
bin/lbs validate # validate skills/cases/troubleshooting structure
bin/lbs index # regenerate skills.index.json
bin/lbs env show # inspect resolved env defaults (redacted)
+7
View File
@@ -5,6 +5,13 @@
"lbs": "./bin/lbs"
},
"scripts": {
"bootstrap": "node scripts/bootstrap-lbs.mjs",
"prepare": "node scripts/bootstrap-lbs.mjs",
"prevalidate": "node scripts/bootstrap-lbs.mjs",
"preindex": "node scripts/bootstrap-lbs.mjs",
"preindex:check": "node scripts/bootstrap-lbs.mjs",
"pretest": "node scripts/bootstrap-lbs.mjs",
"precheck": "node scripts/bootstrap-lbs.mjs",
"lbs": "node src/lbs.ts",
"test": "node test/lbs-cli.test.ts",
"validate": "node src/lbs.ts validate",
+31
View File
@@ -0,0 +1,31 @@
#!/usr/bin/env node
import { chmod, mkdir, readFile, writeFile } from "node:fs/promises";
import { dirname, resolve } from "node:path";
import { fileURLToPath } from "node:url";
const root = resolve(dirname(fileURLToPath(import.meta.url)), "..");
const binDir = resolve(root, "bin");
const lbsPath = resolve(binDir, "lbs");
const wrapper = [
"#!/usr/bin/env bash",
"set -euo pipefail",
"",
'SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"',
'exec node "$SCRIPT_DIR/../src/lbs.ts" "$@"',
"",
].join("\n");
await mkdir(binDir, { recursive: true });
let current = "";
try {
current = await readFile(lbsPath, "utf8");
} catch {
// Missing wrapper is the normal first-run path.
}
if (current !== wrapper) {
await writeFile(lbsPath, wrapper, "utf8");
await chmod(lbsPath, 0o755);
}
+1 -1
View File
@@ -61,7 +61,7 @@ export function repoRoot(start: string): string {
// root of this assets tree). Check it first so that when the tree lives
// inside a larger repo (e.g. LangBot/skills/), we stop at the assets root
// and not at the outer repo's .git/README.md.
if (existsSync(`${current}/skills.index.json`) && existsSync(`${current}/bin/lbs`)) {
if (existsSync(`${current}/skills.index.json`) && existsSync(`${current}/schemas/case.schema.json`)) {
return current;
}
if (existsSync(`${current}/.git`) && existsSync(`${current}/README.md`)) {
+83 -41
View File
@@ -15,6 +15,7 @@ import { commandTroubleSearch } from "../src/commands/trouble.ts";
import { commandValidate } from "../src/commands/validate.ts";
import { commandIndex } from "../src/commands/skill.ts";
import { loadEnv } from "../src/fs.ts";
import { repoRoot } from "../src/cli.ts";
import {
classifyDebugChatResult,
findNewFailureSignal,
@@ -23,6 +24,19 @@ import {
const root = process.cwd();
test("repo root detects the skills tree before generated bin exists", () => {
const tmp = mkdtempSync(join(tmpdir(), "lbs-root-no-bin-"));
try {
mkdirSync(join(tmp, "schemas"), { recursive: true });
mkdirSync(join(tmp, "skills", "langbot-testing"), { recursive: true });
writeFileSync(join(tmp, "skills.index.json"), "{}");
writeFileSync(join(tmp, "schemas", "case.schema.json"), "{}");
assert.equal(repoRoot(join(tmp, "skills", "langbot-testing")), tmp);
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
function ctx(args: string[]): CommandContext {
return { root, args };
}
@@ -75,6 +89,19 @@ function suiteResult(caseId: string, runId: string, status = "pass", evidence =
});
}
function withEnv<T>(values: Record<string, string>, fn: () => T): T {
const previous = new Map(Object.keys(values).map((key) => [key, process.env[key]]));
try {
for (const [key, value] of Object.entries(values)) process.env[key] = value;
return fn();
} finally {
for (const [key, value] of previous) {
if (value === undefined) delete process.env[key];
else process.env[key] = value;
}
}
}
async function captureAsync(fn: () => Promise<number>): Promise<{ code: number; output: string }> {
const originalLog = console.log;
const lines: string[] = [];
@@ -1417,15 +1444,20 @@ test("generic pipeline readiness accepts either URL or name target", () => {
const originalUrl = process.env.LANGBOT_PIPELINE_URL;
const originalName = process.env.LANGBOT_PIPELINE_NAME;
try {
process.env.LANGBOT_PIPELINE_URL = "http://127.0.0.1:3000/home/pipelines?id=only-url";
process.env.LANGBOT_PIPELINE_NAME = "";
withEnv({
LANGBOT_BROWSER_PROFILE: "/tmp/langbot-test-profile",
LANGBOT_CHROMIUM_EXECUTABLE: "/tmp/langbot-test-chromium",
}, () => {
process.env.LANGBOT_PIPELINE_URL = "http://127.0.0.1:3000/home/pipelines?id=only-url";
process.env.LANGBOT_PIPELINE_NAME = "";
const ready = capture(() => commandTestPlan(ctx(["test", "plan", "pipeline-debug-chat", "--json"])));
assert.equal(ready.code, 0);
const plan = JSON.parse(ready.output);
assert.equal(plan.env_readiness.status, "ready");
assert.equal(plan.automation_readiness.status, "ready");
assert.ok(plan.automation_readiness.required.includes("LANGBOT_PIPELINE_URL|LANGBOT_PIPELINE_NAME"));
const ready = capture(() => commandTestPlan(ctx(["test", "plan", "pipeline-debug-chat", "--json"])));
assert.equal(ready.code, 0);
const plan = JSON.parse(ready.output);
assert.equal(plan.env_readiness.status, "ready");
assert.equal(plan.automation_readiness.status, "ready");
assert.ok(plan.automation_readiness.required.includes("LANGBOT_PIPELINE_URL|LANGBOT_PIPELINE_NAME"));
});
process.env.LANGBOT_PIPELINE_URL = "";
process.env.LANGBOT_PIPELINE_NAME = "";
@@ -2174,27 +2206,32 @@ test("local-agent effective prompt case has runnable automation defaults", () =>
});
test("local-agent basic case can setup the local-agent pipeline env", () => {
const result = capture(() => commandTestRun(ctx([
"test",
"run",
"local-agent-basic-debug-chat",
"--dry-run",
"--json",
])));
assert.equal(result.code, 0);
const run = JSON.parse(result.output);
assert.deepEqual(run.setup_automation.map((item: { entry: string }) => item.entry), [
"node:scripts/e2e/ensure-local-agent-pipeline.mjs --write-env",
]);
withEnv({
LANGBOT_BROWSER_PROFILE: "/tmp/langbot-test-profile",
LANGBOT_CHROMIUM_EXECUTABLE: "/tmp/langbot-test-chromium",
}, () => {
const result = capture(() => commandTestRun(ctx([
"test",
"run",
"local-agent-basic-debug-chat",
"--dry-run",
"--json",
])));
assert.equal(result.code, 0);
const run = JSON.parse(result.output);
assert.deepEqual(run.setup_automation.map((item: { entry: string }) => item.entry), [
"node:scripts/e2e/ensure-local-agent-pipeline.mjs --write-env",
]);
const planResult = capture(() => commandTestPlan(ctx(["test", "plan", "local-agent-basic-debug-chat", "--json"])));
assert.equal(planResult.code, 0);
const plan = JSON.parse(planResult.output);
assert.deepEqual(plan.setup_provides_env, [
"LANGBOT_LOCAL_AGENT_PIPELINE_URL",
"LANGBOT_LOCAL_AGENT_PIPELINE_NAME",
]);
assert.equal(plan.automation_readiness.status, "ready");
const planResult = capture(() => commandTestPlan(ctx(["test", "plan", "local-agent-basic-debug-chat", "--json"])));
assert.equal(planResult.code, 0);
const plan = JSON.parse(planResult.output);
assert.deepEqual(plan.setup_provides_env, [
"LANGBOT_LOCAL_AGENT_PIPELINE_URL",
"LANGBOT_LOCAL_AGENT_PIPELINE_NAME",
]);
assert.equal(plan.automation_readiness.status, "ready");
});
});
test("local-agent nonstreaming case disables stream output through automation defaults", () => {
@@ -2355,20 +2392,25 @@ test("AgentRunner QA Debug Chat case uses dedicated pipeline env", () => {
});
test("AgentRunner QA Debug Chat setup automation removes manual readiness", () => {
const planResult = capture(() => commandTestPlan(ctx(["test", "plan", "agent-runner-qa-debug-chat", "--json"])));
assert.equal(planResult.code, 0);
const plan = JSON.parse(planResult.output);
assert.equal(plan.manual_readiness.status, "not_required");
assert.deepEqual(plan.setup_provides_env, [
"LANGBOT_QA_AGENT_RUNNER_PIPELINE_URL",
"LANGBOT_QA_AGENT_RUNNER_PIPELINE_NAME",
]);
assert.equal(plan.automation_readiness.status, "ready");
withEnv({
LANGBOT_BROWSER_PROFILE: "/tmp/langbot-test-profile",
LANGBOT_CHROMIUM_EXECUTABLE: "/tmp/langbot-test-chromium",
}, () => {
const planResult = capture(() => commandTestPlan(ctx(["test", "plan", "agent-runner-qa-debug-chat", "--json"])));
assert.equal(planResult.code, 0);
const plan = JSON.parse(planResult.output);
assert.equal(plan.manual_readiness.status, "not_required");
assert.deepEqual(plan.setup_provides_env, [
"LANGBOT_QA_AGENT_RUNNER_PIPELINE_URL",
"LANGBOT_QA_AGENT_RUNNER_PIPELINE_NAME",
]);
assert.equal(plan.automation_readiness.status, "ready");
const suiteResult = capture(() => commandSuitePlan(ctx(["suite", "plan", "agent-runner-release-gate", "--json"])));
assert.equal(suiteResult.code, 0);
const suite = JSON.parse(suiteResult.output);
assert.ok(!suite.readiness.manual_check_cases.includes("agent-runner-qa-debug-chat"));
const suiteResult = capture(() => commandSuitePlan(ctx(["suite", "plan", "agent-runner-release-gate", "--json"])));
assert.equal(suiteResult.code, 0);
const suite = JSON.parse(suiteResult.output);
assert.ok(!suite.readiness.manual_check_cases.includes("agent-runner-qa-debug-chat"));
});
});
test("ACP AgentRunner Debug Chat case setups the ACP pipeline env", () => {
+7 -1
View File
@@ -84,7 +84,13 @@ class CommandManager:
privilege = 1
if f'{query.launcher_type.value}_{query.launcher_id}' in self.ap.instance_config.data['admins']:
admins = self.ap.instance_config.data['admins']
launcher_session_id = f'{query.launcher_type.value}_{query.launcher_id}'
sender_session_id = f'person_{query.sender_id}'
# 兼容老版本匹配 launcher_session_id(群管理: group_xxx 私聊管理: person_xxx
# 新实现匹配 sender_session_id(个人管理员: person_xxx,在任何群聊中生效)
if launcher_session_id in admins or sender_session_id in admins:
privilege = 2
ctx = command_context.ExecuteContext(
@@ -23,7 +23,13 @@ class CommandHandler(handler.MessageHandler):
privilege = 1
if f'{query.launcher_type.value}_{query.launcher_id}' in self.ap.instance_config.data['admins']:
admins = self.ap.instance_config.data['admins']
launcher_session_id = f'{query.launcher_type.value}_{query.launcher_id}'
sender_session_id = f'person_{query.sender_id}'
# 兼容老版本匹配 launcher_session_id(群管理: group_xxx 私聊管理: person_xxx
# 新实现匹配 sender_session_id(个人管理员: person_xxx,在任何群聊中生效)
if launcher_session_id in admins or sender_session_id in admins:
privilege = 2
spt = command_text.split(' ')
@@ -646,7 +646,7 @@ export default function ModelsPanel({
</PanelBody>
<Dialog open={providerFormOpen} onOpenChange={setProviderFormOpen}>
<DialogContent className="w-[600px] p-6">
<DialogContent className="w-full max-w-[calc(100%-2rem)] p-4 sm:max-w-[600px] sm:p-6">
<DialogHeader>
<DialogTitle>
{editingProviderId
@@ -38,7 +38,12 @@ export function PanelBody({
children: React.ReactNode;
}) {
return (
<div className={cn('min-h-0 flex-1 overflow-auto px-6 py-5', className)}>
<div
className={cn(
'min-h-0 flex-1 overflow-auto px-3 py-4 sm:px-6 sm:py-5',
className,
)}
>
{children}
</div>
);
@@ -109,10 +109,10 @@ export default function MonitoringFilters({
};
return (
<div className="flex flex-wrap items-center gap-6">
<div className="flex w-full flex-col gap-3 sm:w-auto sm:flex-row sm:flex-wrap sm:items-center sm:gap-6">
{/* Bot Filter */}
<div className="flex items-center gap-2">
<label className="text-sm font-medium text-foreground whitespace-nowrap">
<label className="w-20 shrink-0 text-sm font-medium text-foreground sm:w-auto sm:whitespace-nowrap">
{t('monitoring.filters.bot')}
</label>
<Select
@@ -120,7 +120,7 @@ export default function MonitoringFilters({
onValueChange={handleBotChange}
disabled={loadingBots}
>
<SelectTrigger className="h-9 w-[140px]">
<SelectTrigger className="h-9 w-full sm:w-[140px]">
<SelectValue
placeholder={
loadingBots
@@ -144,7 +144,7 @@ export default function MonitoringFilters({
{/* Pipeline Filter */}
<div className="flex items-center gap-2">
<label className="text-sm font-medium text-foreground whitespace-nowrap">
<label className="w-20 shrink-0 text-sm font-medium text-foreground sm:w-auto sm:whitespace-nowrap">
{t('monitoring.filters.pipeline')}
</label>
<Select
@@ -152,7 +152,7 @@ export default function MonitoringFilters({
onValueChange={handlePipelineChange}
disabled={loadingPipelines}
>
<SelectTrigger className="h-9 w-[140px]">
<SelectTrigger className="h-9 w-full sm:w-[140px]">
<SelectValue
placeholder={
loadingPipelines
@@ -176,11 +176,11 @@ export default function MonitoringFilters({
{/* Time Range Filter */}
<div className="flex items-center gap-2">
<label className="text-sm font-medium text-foreground whitespace-nowrap">
<label className="w-20 shrink-0 text-sm font-medium text-foreground sm:w-auto sm:whitespace-nowrap">
{t('monitoring.filters.timeRange')}
</label>
<Select value={timeRange} onValueChange={handleTimeRangeChange}>
<SelectTrigger className="h-9 w-[150px]">
<SelectTrigger className="h-9 w-full sm:w-[150px]">
<SelectValue />
</SelectTrigger>
<SelectContent>
+17 -17
View File
@@ -270,7 +270,7 @@ function MonitoringPageContent() {
{/* Filters and Refresh Button - Sticky */}
<div className="sticky top-0 z-10 -mt-1 pb-5 pt-1 bg-background">
<div>
<div className="flex flex-wrap items-center justify-between gap-4 p-4 bg-card rounded-xl border">
<div className="flex flex-col gap-3 p-3 bg-card rounded-xl border sm:flex-row sm:flex-wrap sm:items-center sm:justify-between sm:gap-4 sm:p-4">
<MonitoringFilters
selectedBots={filterState.selectedBots}
selectedPipelines={filterState.selectedPipelines}
@@ -285,7 +285,7 @@ function MonitoringPageContent() {
variant="outline"
size="sm"
onClick={handleRefresh}
className="shadow-sm flex-shrink-0"
className="flex-1 shadow-sm sm:flex-shrink-0 sm:flex-none"
>
<RefreshCw className="w-4 h-4 mr-2" />
{t('monitoring.refreshData')}
@@ -312,27 +312,27 @@ function MonitoringPageContent() {
onValueChange={setActiveTab}
className="w-full"
>
<div className="px-6 pt-4">
<TabsList className="h-12 p-1">
<TabsTrigger value="messages" className="px-6 py-2">
<div className="px-3 pt-4 sm:px-6">
<TabsList className="h-12 w-full justify-start gap-1 overflow-x-auto p-1 sm:w-auto">
<TabsTrigger value="messages" className="px-3 py-2 sm:px-6">
{t('monitoring.tabs.messages')}
</TabsTrigger>
<TabsTrigger value="modelCalls" className="px-6 py-2">
<TabsTrigger value="modelCalls" className="px-3 py-2 sm:px-6">
{t('monitoring.tabs.modelCalls')}
</TabsTrigger>
<TabsTrigger value="tokens" className="px-6 py-2">
<TabsTrigger value="tokens" className="px-3 py-2 sm:px-6">
{t('monitoring.tabs.tokens')}
</TabsTrigger>
<TabsTrigger value="feedback" className="px-6 py-2">
<TabsTrigger value="feedback" className="px-3 py-2 sm:px-6">
{t('monitoring.tabs.feedback')}
</TabsTrigger>
<TabsTrigger value="errors" className="px-6 py-2">
<TabsTrigger value="errors" className="px-3 py-2 sm:px-6">
{t('monitoring.tabs.errors')}
</TabsTrigger>
</TabsList>
</div>
<TabsContent value="messages" className="p-6 m-0">
<TabsContent value="messages" className="p-3 m-0 sm:p-6">
<div>
{loading && (
<div className="py-12 flex justify-center">
@@ -362,7 +362,7 @@ function MonitoringPageContent() {
>
{/* Message Header - Always Visible */}
<div
className="p-5 cursor-pointer hover:bg-accent transition-colors"
className="p-3 cursor-pointer hover:bg-accent transition-colors sm:p-5"
onClick={() => toggleMessageExpand(msg.id)}
>
<div className="flex items-start justify-between">
@@ -466,7 +466,7 @@ function MonitoringPageContent() {
</div>
</TabsContent>
<TabsContent value="modelCalls" className="p-6 m-0">
<TabsContent value="modelCalls" className="p-3 m-0 sm:p-6">
<div>
{loading && (
<div className="py-12 flex justify-center">
@@ -482,7 +482,7 @@ function MonitoringPageContent() {
{data.modelCalls.map((call) => (
<div
key={call.id}
className="border rounded-xl p-5 transition-all duration-200"
className="border rounded-xl p-3 transition-all duration-200 sm:p-5"
>
<div className="flex justify-between items-start mb-3">
<div className="flex-1">
@@ -672,7 +672,7 @@ function MonitoringPageContent() {
</div>
</TabsContent>
<TabsContent value="tokens" className="p-6 m-0">
<TabsContent value="tokens" className="p-3 m-0 sm:p-6">
<TokenMonitoring
botIds={
filterState.selectedBots.length > 0
@@ -690,7 +690,7 @@ function MonitoringPageContent() {
/>
</TabsContent>
<TabsContent value="feedback" className="p-6 m-0">
<TabsContent value="feedback" className="p-3 m-0 sm:p-6">
<div>
{loading && (
<div className="py-12 flex justify-center">
@@ -722,7 +722,7 @@ function MonitoringPageContent() {
</div>
</TabsContent>
<TabsContent value="errors" className="p-6 m-0">
<TabsContent value="errors" className="p-3 m-0 sm:p-6">
<div>
{loading && (
<div className="py-12 flex justify-center">
@@ -739,7 +739,7 @@ function MonitoringPageContent() {
>
{/* Error Header - Always Visible */}
<div
className="p-5 cursor-pointer hover:bg-red-50 dark:hover:bg-red-950/50 transition-colors bg-red-50/50 dark:bg-red-950/30"
className="p-3 cursor-pointer hover:bg-red-50 dark:hover:bg-red-950/50 transition-colors bg-red-50/50 dark:bg-red-950/30 sm:p-5"
onClick={() => toggleErrorExpand(error.id)}
>
<div className="flex items-start justify-between">
@@ -90,7 +90,7 @@ export default function PluginLogs({
return (
<div className="flex h-full flex-col">
<div className="flex shrink-0 flex-wrap items-center gap-2 px-6 pb-3">
<div className="flex shrink-0 flex-wrap items-center gap-2 px-1 pb-3 sm:px-6">
<Select value={level} onValueChange={setLevel}>
<SelectTrigger className="h-8 w-[130px]">
<SelectValue />
@@ -132,7 +132,7 @@ export default function PluginLogs({
<div
ref={scrollRef}
onScroll={handleScroll}
className="min-h-0 flex-1 overflow-auto bg-gray-50 px-6 py-3 font-mono text-xs leading-relaxed dark:bg-gray-900/40"
className="min-h-0 flex-1 overflow-auto bg-gray-50 px-3 py-3 font-mono text-xs leading-relaxed dark:bg-gray-900/40 sm:px-6"
>
{logs.length === 0 ? (
<div className="py-8 text-center text-sm text-gray-500 dark:text-gray-400">