Skip to content

Simplify analyzer UI to single-input, core-result flow#54

Merged
rad1092 merged 1 commit intomainfrom
codex-7h9k14
Feb 15, 2026
Merged

Simplify analyzer UI to single-input, core-result flow#54
rad1092 merged 1 commit intomainfrom
codex-7h9k14

Conversation

@rad1092
Copy link
Copy Markdown
Owner

@rad1092 rad1092 commented Feb 15, 2026

Motivation

  • Reduce cognitive load by making the default UI focused on a single request input, one execute action, and a single core result card.
  • Keep advanced debug and file/advanced features available but hidden by default to avoid clutter and preserve backward-compatible element IDs for existing logic and tests.
  • Expose the generated prompt/code only when the user explicitly requests it via a recognizable phrase (코드도 보여줘).

Description

  • Reworked bitnet_tools/ui/index.html to present a single primary textarea#question, a single #quickAnalyzeBtn and a focused result card (#summary), moved file/csv controls into a collapsed <details> section, and placed advanced debug outputs inside collapsed details.
  • Updated bitnet_tools/ui/app.js to add UI.rawJson and UI.codeDetails references, added CODE_REQUEST_PATTERN, shouldShowCodeBlock, formatCoreSummary, and renderPrimaryResult helpers, and wired analyze flow to render only the concise core result by default while showing the prompt block conditionally.
  • Adjusted bitnet_tools/ui/styles.css to style the simplified panel and result card (.simple-panel, .result-card) and slightly widen the container for the new layout.
  • Preserved advanced element IDs and a hidden advanced-only section so existing APIs and tests keep working.

Testing

  • Ran pytest -q tests/test_ui_contract.py tests/test_web.py and all tests passed (17 passed).
  • Executed an automated Playwright script to load the UI and capture a screenshot (artifacts/simple-ui.png), which ran successfully.

Codex Task

@rad1092 rad1092 merged commit f6856fa into main Feb 15, 2026
4 checks passed
@rad1092 rad1092 deleted the codex-7h9k14 branch February 15, 2026 07:43
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 08090f8718

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

<h3>드릴다운(근거 데이터)</h3>
<pre id="insightDrilldown">인사이트를 선택하면 근거 데이터가 표시됩니다.</pre>

<section class="advanced-only" hidden>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Remove permanent hidden flag from advanced controls section

The new <section class="advanced-only" hidden> makes all advanced controls permanently non-rendered, because setMode in app.js only toggles style.display for .advanced-only elements and never clears the hidden attribute. In this commit, that hidden section now contains both mode-toggle buttons and the multi/dashboard/geo inputs, so users cannot enter advanced mode or supply inputs required by multi/visualize flows from the UI.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant