Skip to content

docs(design): scrub the owner's name, a real log path and a prefilled password from the Refined UI mockups (#18238) - #18250

Merged
mrveiss merged 2 commits into
mainfrom
issue-18238-scrub-leak
Oct 11, 2026
Merged

mrveiss merged 2 commits into
mainfrom
issue-18238-scrub-leak

Conversation

@mrveiss

@mrveiss mrveiss commented Oct 11, 2026

Copy link
Copy Markdown
Owner

Refs #18238

Thinking Path

#18226 published the Refined UI mockups with a scrub that missed four lines: the owner's first name (with diacritics) in a greeting and a table fixture, a real log install path, and a password prefilled into the Login mock. They are public on main now, so this ships alone ahead of the full canvas refresh (#18238), which rides on the step-1 PR.

What Changed

  • docs/design/mockups/refined-ui/screens/Main.dc.html:176: the greeting now uses the placeholder first name "Jane".
  • docs/design/mockups/refined-ui/screens/ComponentsTable.dc.html:678: ME is now 'Jane D.'.
  • docs/design/mockups/refined-ui/screens/KnowledgeMcpResources.dc.html:239-240: the log URI is now file:///var/log/app/backend.log.
  • docs/design/mockups/refined-ui/screens/Login.dc.html:168: the password input's prefilled value is removed. The placeholder stays.

Docs only, 4 files, +5/-5.

Verification

  • A case-insensitive re-scan of docs/design/mockups/ for martins|mārtiņš|veiss|mrveiss, the prefilled password, /var/log/autobot, /opt/autobot, private IPv4 (10/8, 172.16/12, 192.168/16) and .local/.lan/.internal: 0 hits.
  • The remaining autobot-labs strings are the fictional org on .example hosts. The mv matches are JavaScript variable names.
  • pre-commit passed on commit (detect-secrets, repo conventions). The branch was 0 behind origin/main at push.

Model Used

Claude Opus 5.5 (session), with Sonnet subagents for the canvas diff.

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 26 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: mrveiss/AutoBot-AI/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: a92ad4ca-3684-44e0-9462-aa54a68c84c7

📥 Commits

Reviewing files that changed from the base of the PR and between 1c8f70b and 979de09.


📒 Files selected for processing (4)
  • docs/design/mockups/refined-ui/screens/ComponentsTable.dc.html
  • docs/design/mockups/refined-ui/screens/KnowledgeMcpResources.dc.html
  • docs/design/mockups/refined-ui/screens/Login.dc.html
  • docs/design/mockups/refined-ui/screens/Main.dc.html

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant