Skip to content

Release 2026 beta jun - #324

Merged
vitonsky merged 4 commits into
masterfrom
release-2026-beta-jun
Jun 16, 2026
Merged

Release 2026 beta jun#324
vitonsky merged 4 commits into
masterfrom
release-2026-beta-jun

Conversation

@vitonsky

@vitonsky vitonsky commented Jun 15, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Bug Fixes
    • Prevented the browser’s default context menu from appearing when using the editor’s custom context menu.
    • Fixed alignment of the object properties editor header layout.
  • Chores
    • Updated the app version to 0.2.0 and refreshed Flatpak release metadata.
    • Updated the Flatpak module to download v0.2.0 and adjusted the build/repo workflow to support the full release sequence more reliably.

@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

ContextMenuPlugin now calls evt.preventDefault() to suppress the browser's native context menu. ObjectPropertiesEditor's Card.Header receives an explicit flexDirection="row" prop. The app version is bumped to 0.2.0, and the Flatpak build infrastructure is refactored to isolate artifacts under /tmp/deepink-flatpak, with updated build state directory handling, repository initialization, and orchestration targets for coordinated release workflows.

Changes

0.2.0 Release - UI Fixes and Flatpak Infrastructure

Layer / File(s) Summary
ContextMenu event prevention and UI layout
packages/app/src/features/NoteEditor/RichEditor/plugins/ContextMenu/ContextMenuPlugin.tsx, packages/app/src/features/NoteEditor/RichEditor/plugins/ContextMenu/components/ObjectPropertiesEditor.tsx
ContextMenuPlugin calls evt.preventDefault() on contextmenu events to block the browser's native menu. ObjectPropertiesEditor's Card.Header is given explicit flexDirection="row" to enforce horizontal layout.
Flatpak build directory isolation and targets
packages/app/scripts/flathub/Makefile
Variables are refactored to isolate build and repository artifacts under /tmp/deepink-flatpak. SUBDIR and BUILD_STATE_DIR are introduced; BUILD_DIR and REPO_DIR use the new base directory. The dev-install and repo-export targets now pass --state-dir=$(BUILD_STATE_DIR).
Flatpak repository pull and build orchestration
packages/app/scripts/flathub/Makefile
repo-pull is rewritten to create the repo directory, initialize an OSTree archive-z2 repository locally, and copy remote contents via rclone copy. A new repo-build target orchestrates repo-pull, repo-update, and repo-push.
Version bump and release metadata
packages/app/package.json, packages/app/scripts/flathub/app.deepink.Deepink.metainfo.xml, packages/app/scripts/flathub/app.deepink.Deepink.yaml
Package version bumped to 0.2.0. Flatpak AppStream metadata and module source are updated to v0.2.0 with new checksums and release date.
Build workflow documentation
packages/app/scripts/flathub/README.md
README updated to document release workflow: make repo-update repo-push for updates, with make repo-build automating the full sequence.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • DeepinkApp/deepink#290: Both PRs modify the Flatpak packaging workflow in packages/app/scripts/flathub/Makefile and update associated Flatpak metadata/docs/releases.
  • DeepinkApp/deepink#288: Both PRs modify the Flatpak publishing pipeline in packages/app/scripts/flathub/Makefile by adjusting repo update/push/pull-related targets and variables.
  • DeepinkApp/deepink#289: Both PRs modify packages/app/scripts/flathub/Makefile, specifically the Flatpak repo workflow and targets.

Poem

🐇 A menu fixed, a flex row set,
The version climbs to point-two yet,
Build artifacts tucked in /tmp so neat,
OSTree repos with checksums sweet,
The rabbit hops—release complete! 🥕

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Release 2026 beta jun' is vague and does not clearly convey the specific changes in this pull request, which primarily involve releasing version 0.2.0 with various feature updates and build improvements. Consider using a more descriptive title like 'Release v0.2.0' or 'Release v0.2.0 with context menu and build improvements' to clearly indicate the version being released and main changes.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch release-2026-beta-jun

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 and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/app/package.json`:
- Line 6: The Flatpak distribution metadata is out of sync with the version bump
from 0.1.3 to 0.2.0 in package.json. Update the release version in
app.deepink.Deepink.metainfo.xml from "0.1.3-preview.1" to "0.2.0" in the
release element. In app.deepink.Deepink.yaml, update the source URL from the
v0.1.3 release artifact to the corresponding v0.2.0 release artifact path
(changing "v0.1.3/deepink-linux-x64-0.1.3.zip" to the v0.2.0 equivalent), and
update the sha256 checksum to match the new v0.2.0 release archive. These
changes ensure the Flatpak builder will fetch and build against the correct
version.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d0ddf719-3908-4456-8de7-df71467ff636

📥 Commits

Reviewing files that changed from the base of the PR and between 234ae2a and 62178ef.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (3)
  • packages/app/package.json
  • packages/app/src/features/NoteEditor/RichEditor/plugins/ContextMenu/ContextMenuPlugin.tsx
  • packages/app/src/features/NoteEditor/RichEditor/plugins/ContextMenu/components/ObjectPropertiesEditor.tsx

Comment thread packages/app/package.json

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/app/scripts/flathub/Makefile`:
- Around line 39-42: The rclone copy command on line 42 only adds or updates
files from the remote to the local repository but does not remove stale local
files that are no longer in the remote. This causes the subsequent rclone sync
operation on line 34 to republish these stale local artifacts back to the
remote, drifting the remote state. Replace rclone copy with rclone sync on line
42 to ensure the local repository exactly matches the remote state, removing any
stale files before the later sync operation pushes to remote.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 9a54e95d-ec4d-4f8d-ab79-55539a3d5b7a

📥 Commits

Reviewing files that changed from the base of the PR and between 62178ef and 5aa2fbf.

📒 Files selected for processing (4)
  • packages/app/scripts/flathub/Makefile
  • packages/app/scripts/flathub/README.md
  • packages/app/scripts/flathub/app.deepink.Deepink.metainfo.xml
  • packages/app/scripts/flathub/app.deepink.Deepink.yaml
✅ Files skipped from review due to trivial changes (3)
  • packages/app/scripts/flathub/README.md
  • packages/app/scripts/flathub/app.deepink.Deepink.yaml
  • packages/app/scripts/flathub/app.deepink.Deepink.metainfo.xml

Comment thread packages/app/scripts/flathub/Makefile
@vitonsky
vitonsky merged commit c5741b9 into master Jun 16, 2026
6 checks passed
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