Skip to content

docs(wallpaper): explain Hidamari movie setup - #1323

Merged
castrojo merged 1 commit into
projectbluefin:mainfrom
arteeh:fix/hidamari-setup
Oct 1, 2026
Merged

castrojo merged 1 commit into
projectbluefin:mainfrom
arteeh:fix/hidamari-setup

Conversation

@arteeh

@arteeh arteeh commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

What does this change?

Adds a README-linked Hidamari setup guide explaining how to copy the shipped Bluefin movie into the user's XDG Videos/Hidamari folder and select it in Local Video. The command respects localized paths and asks before replacing an existing file. Updates the maintainer skill with the same sandbox constraint.

Why?

PR #1207 shipped the movie but deliberately left #1158 open because Hidamari cannot discover the system copy. Its review explicitly offered documented copy instructions as a resolution. This supplies that manual setup path, including troubleshooting and removal instructions.

Closes #1158.

Validation

  • just check passed.
  • pre-commit run --all-files passed, including the new guide.
  • Executed the documented copy snippet with the repository movie as its source and a temporary localized Videos path containing spaces; verified byte-for-byte copying and that declining replacement preserves existing contents.
  • Checked the Flathub manifest's xdg-videos/Hidamari:create permission.
  • Hidamari GUI playback was not tested in this environment. This is documentation only; no automatic first-run behavior is added.

Checklist

  • Conventional Commits title
  • Local lint and pre-commit checks pass
  • Relevant skill updated and internal links validated
  • CI results pending after PR creation

— hive: backend=codex model=gpt-6-astra effort=medium


🐝 Hive Agent: contributor | SHA: 6364878

Document copying the shipped movie into the XDG Videos/Hidamari folder so the Flatpak can discover it. Link the guide from the README and record the sandbox constraint in the maintainer skill.

Hive-Run: projectbluefin#1158
Hive-Plan: hidamari-user-setup
Hive-Spec: projectbluefin#1158#hidamari-accessibility
Assisted-by: GPT-6 Astra via Codex
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: arteeh <arteeh@users.noreply.github.com>
@arteeh
arteeh requested review from a team and repires as code owners September 30, 2026 19:04

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

correctness

  • (low) docs/hidamari.md:13-14 — xdg-user-dir VIDEOS never prints an empty string: when XDG_VIDEOS_DIR is unconfigured it falls back to $HOME, so the test -n guard cannot trigger and the snippet silently creates ~/Hidamari. The Flatpak's --filesystem=xdg-videos/Hidamari:create does not map to $HOME in that case, so the copy lands where Hidamari cannot see it. Consider guarding test "$videos_dir" != "$HOME" (or telling the user to run xdg-user-dirs-update first). Rare on Bluefin GNOME logins, so non-blocking.

docs-currency

  • (low) docs/skills/hidamari-wallpaper.md:4 — front matter still says last_updated: "2026-09-24" though this PR adds a new section (lines 33-42). Bump it so the skill index reflects the change.

No findings from: security, intent-alignment, style.

Confidence: 5/5 (safe)

— hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88

@Danathar Danathar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed at 6364878. This adds a short guide for using the movie Bluefin ships with Hidamari, plus a README link and a maintainer note. The movie exists at the path the guide copies from (system_files/bluefin/usr/share/backgrounds/bluefin/bluefin-hidamari.webm), and the Flathub manifest grants xdg-videos/Hidamari, so copying instead of symlinking is right. The doc link, skill front matter and skill index checks pass locally, and CI is green.

Two small suggestions, not blocking:

  • xdg-user-dir VIDEOS prints $HOME when no Videos folder is configured, not an empty string, so the test -n guard never fires and the copy would land in ~/Hidamari, which the Flatpak can't see. Checking != "$HOME" would catch that.
  • last_updated in docs/skills/hidamari-wallpaper.md still says 2026-09-24.

Not tried: the steps in a live Hidamari install.

@castrojo
castrojo enabled auto-merge October 1, 2026 02:28

@castrojo castrojo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, docs update

@castrojo
castrojo added this pull request to the merge queue Oct 1, 2026
Merged via the queue into projectbluefin:main with commit b881ec7 Oct 1, 2026
5 checks passed
dtg01100 pushed a commit to dtg01100/common that referenced this pull request Oct 3, 2026
## What does this change?

Adds a README-linked Hidamari setup guide explaining how to copy the
shipped Bluefin movie into the user's XDG Videos/Hidamari folder and
select it in Local Video. The command respects localized paths and asks
before replacing an existing file. Updates the maintainer skill with the
same sandbox constraint.

## Why?

PR projectbluefin#1207 shipped the movie but deliberately left projectbluefin#1158 open because
Hidamari cannot discover the system copy. Its review explicitly offered
documented copy instructions as a resolution. This supplies that manual
setup path, including troubleshooting and removal instructions.

Closes projectbluefin#1158.

## Validation

- `just check` passed.
- `pre-commit run --all-files` passed, including the new guide.
- Executed the documented copy snippet with the repository movie as its
source and a temporary localized Videos path containing spaces; verified
byte-for-byte copying and that declining replacement preserves existing
contents.
- Checked the Flathub manifest's `xdg-videos/Hidamari:create`
permission.
- Hidamari GUI playback was not tested in this environment. This is
documentation only; no automatic first-run behavior is added.

## Checklist

- [x] Conventional Commits title
- [x] Local lint and pre-commit checks pass
- [x] Relevant skill updated and internal links validated
- [ ] CI results pending after PR creation

— hive: backend=codex model=gpt-6-astra effort=medium

---
🐝 **Hive Agent**: `contributor` | **SHA:** `6364878`

Signed-off-by: arteeh <arteeh@users.noreply.github.com>
Co-authored-by: arteeh <arteeh@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

Make a movie for Hidamari

3 participants