Skip to content

feat: improve line prefix display with box drawing character#274

Open
LinruoT wants to merge 1 commit intoNanmiCoder:mainfrom
LinruoT:line-prefix-improvement
Open

feat: improve line prefix display with box drawing character#274
LinruoT wants to merge 1 commit intoNanmiCoder:mainfrom
LinruoT:line-prefix-improvement

Conversation

@LinruoT
Copy link
Copy Markdown

@LinruoT LinruoT commented May 4, 2026

Summary

  • Replace tab character with box drawing character for line number prefix
  • Update regex pattern to handle new prefix format

Motivation

DeepSeekV4 often confuses the tab separator in line number prefixes with actual tab characters in file content when updating code files, leading to update errors. Using as the line number prefix separator eliminates this confusion.

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label May 4, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

PR quality triage

Changed areas: area:cli-core

CLI core policy: Blocked by policy until a maintainer applies allow-cli-core-change and approves the PR.

CLI core files:

  • src/tools/FileEditTool/prompt.ts
  • src/utils/file.ts

Expected checks:

  • change-policy
  • server-checks

Test coverage signals:

  • Agent/model runtime path changed: use mock/request-shape tests in PR and maintainer live-model smoke before release.

Risk notes:

  • No special risk notes from changed paths.

Hard merge gates still come from GitHub Actions, not AI review.

Dosu handoff: Dosu can be used as the AI reviewer for risk explanation, missing-test prompts, and maintainer Q&A. If it does not comment automatically from the PR template, ask:

@dosubot review this PR for changed-area risk, missing tests, docs impact, desktop startup risk, and CLI core impact.

@dosubot dosubot Bot added the enhancement New feature or request label May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:cli-core enhancement New feature or request needs-maintainer-approval size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant