Skip to content

feat: add tag chip visibility setting for 1.8.0 - #90

Merged
easyhooon merged 2 commits into
mainfrom
chore/bump-version-1.8.0
Sep 2, 2026
Merged

easyhooon merged 2 commits into
mainfrom
chore/bump-version-1.8.0

Conversation

@easyhooon

@easyhooon easyhooon commented Aug 10, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Add a persisted Settings toggle for message-row bridge tag chips
  • Keep the top tag filter available when row chips are hidden
  • Bump Dari artifacts and installation examples from 1.7.0 to 1.8.0

Motivation

Long bridge tags can reduce the space available for handler names. This keeps tag filtering available while allowing developers to hide only the per-message chip. Version 1.8.0 also includes the Kotlin Multiplatform core support merged in #88.

Verification

  • ./gradlew ktlintCheck test
  • Commit hook: detekt and ktlint checks
  • Tag chip preference default and persistence unit tests

Summary by CodeRabbit

  • New Features

    • Added a setting to show or hide bridge tag chips beside handler names.
    • The preference is saved and restored across sessions.
    • Added a public API for controlling tag chip visibility.
  • Documentation

    • Updated setup instructions and API references for the latest release version.
  • Tests

    • Added coverage for default visibility and persistence behavior.

배포 아티팩트와 영문·한글 설치 문서의 버전을 함께 맞춘다.
@coderabbitai

coderabbitai Bot commented Aug 10, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 7af1a963-b961-4e46-bc7d-ee1421f53c65

📥 Commits

Reviewing files that changed from the base of the PR and between 9189dd9 and 0cca8a7.

📒 Files selected for processing (11)
  • README.md
  • dari-noop/src/main/kotlin/com/easyhooon/dari/Dari.kt
  • dari/src/main/kotlin/com/easyhooon/dari/Dari.kt
  • dari/src/main/kotlin/com/easyhooon/dari/data/DariPreferences.kt
  • dari/src/main/kotlin/com/easyhooon/dari/ui/DariActivity.kt
  • dari/src/main/kotlin/com/easyhooon/dari/ui/components/MessageListItem.kt
  • dari/src/main/kotlin/com/easyhooon/dari/ui/components/SettingsBottomSheet.kt
  • dari/src/test/kotlin/com/easyhooon/dari/data/DariPreferencesTest.kt
  • documentation/content/docs/ko/setup.mdx
  • documentation/content/docs/setup.mdx
  • gradle/libs.versions.toml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The PR adds a persisted showTagChip setting. The settings sheet changes the value, and message items conditionally render bridge tag chips. Tests cover defaults and persistence. Documentation and dependency versions update to 1.8.0.

Changes

Bridge tag chip visibility

Layer / File(s) Summary
Preference and public API
dari/src/main/kotlin/com/easyhooon/dari/data/DariPreferences.kt, dari/src/main/kotlin/com/easyhooon/dari/Dari.kt, dari-noop/src/main/kotlin/com/easyhooon/dari/Dari.kt
DariPreferences stores show_tag_chip with a default value of true. Dari exposes the setter. The no-op implementation provides the matching method.
Settings and message rendering
dari/src/main/kotlin/com/easyhooon/dari/ui/DariActivity.kt, dari/src/main/kotlin/com/easyhooon/dari/ui/components/MessageListItem.kt, dari/src/main/kotlin/com/easyhooon/dari/ui/components/SettingsBottomSheet.kt
The settings sheet controls tag chip visibility. DariActivity passes the setting to MessageListItem, which renders tags only when enabled.
Validation and release documentation
dari/src/test/kotlin/com/easyhooon/dari/data/DariPreferencesTest.kt, README.md, gradle/libs.versions.toml, documentation/content/docs/setup.mdx, documentation/content/docs/ko/setup.mdx
Tests verify the default and persisted values. The README documents the API. Dependency references update to version 1.8.0.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to 0cca8

The PR adds an optional setting to hide per-message bridge-tag chips while keeping tag filtering available; no actionable merge-blocking risk remains beyond normal checks and review.

Sequence Diagram(s)

sequenceDiagram
  participant SettingsBottomSheet
  participant DariActivity
  participant Dari
  participant DariPreferences
  participant MessageListItem
  SettingsBottomSheet->>DariActivity: change showTagChip
  DariActivity->>Dari: setShowTagChip(show)
  Dari->>DariPreferences: persist setting
  DariPreferences-->>DariActivity: emit showTagChip state
  DariActivity->>MessageListItem: pass showTagChip
  MessageListItem-->>DariActivity: show or hide bridge tag chip
Loading

Poem

A rabbit flips the bridge tag switch,
The chips hide fast or glow bright.
Preferences keep the chosen state,
Tests watch it through the night.
“Hop,” says the rabbit, “release is right!”

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 13.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 15 functions across 7 files. (4 skipped: … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main change: adding a tag chip visibility setting. The 1.8.0 reference is also consistent with the version updates in the pull request.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

Docstring coverage is 13.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 15 functions across 7 files. (4 skipped: 4 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-version-1.8.0

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.

긴 핸들러 이름을 온전히 확인할 수 있도록 목록 태그 칩 표시 여부를 저장한다.
@easyhooon easyhooon changed the title chore: bump version to 1.8.0 feat: add tag chip visibility setting for 1.8.0 Sep 2, 2026
@easyhooon
easyhooon merged commit 7252b58 into main Sep 2, 2026
4 checks passed
@easyhooon
easyhooon deleted the chore/bump-version-1.8.0 branch September 2, 2026 06:47
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