Skip to content

deps: bump HLab.Core to drop the dead CryptService - #543

Merged
mgth merged 1 commit into
masterfrom
bump-hlab-core
Aug 16, 2026
Merged

deps: bump HLab.Core to drop the dead CryptService#543
mgth merged 1 commit into
masterfrom
bump-hlab-core

Conversation

@mgth

@mgth mgth commented Aug 15, 2026

Copy link
Copy Markdown
Owner

Picks up mgth/HLab.Core#6, which removes CryptService — TripleDES with a hardcoded key and IV, referenced from nowhere in either repository. The smart-TV token encryption in #542 deliberately did not build on it.

One line, one submodule pointer:

Submodule HLab.Core 7b6aeec..7c6ed67:
  > CryptService: drop the unused TripleDES helper

Why 7c6ed67 and not the master tip

7c6ed67 is the old pin plus that deletion and nothing else, and it is permanently reachable through the #6 merge commit, so it does not depend on the PR branch surviving.

HLab.Core master has since moved six commits further:

8b5f9c8 Restore HLab.Mvvm.Application and finish net10 alignment
70cd582 Bootstrapper: surface stalled bootloaders on stderr
6f17c90 ApplicationBootloader: default MainViewMode to DefaultViewMode
0494779 MessageBus: root subscribed delegates
1a5d56a MvvmService: enregistrer les generiques fermes au runtime
e5b2388 DocumentService: logger les echecs d ouverture de document

That is a restored HLab.Mvvm.Application project plus MessageBus and MvvmService behaviour changes — 447 insertions across 36 files. Adopting it is a separate decision with its own testing, not something to carry in behind a one-file deletion. Tracking the tip here would have done exactly that.

Verification

dotnet build LittleBigMouse-Linux.slnf succeeds with HLab.Core/CryptService.cs actually absent from the working tree.

Independent of #542 — disjoint files, both branch off master, they merge in either order.

🤖 Generated with Claude Code

Picks up mgth/HLab.Core#6, which removes CryptService -- TripleDES with a
hardcoded key and IV, referenced from nowhere in either repository. The
smart-TV token encryption added alongside deliberately did not build on it.

Pinned to 7c6ed67 rather than the master tip on purpose. That commit is the
old pin plus the deletion and nothing else, and it is permanently reachable
through the #6 merge. HLab.Core master has since moved six commits further
-- a restored HLab.Mvvm.Application project, MessageBus and MvvmService
changes, 447 insertions across 36 files -- and adopting that is a separate
decision with its own testing, not something to smuggle in behind a
one-file deletion.

Co-Authored-By: Claude <noreply@anthropic.com>
@mgth
mgth merged commit a98cf77 into master Aug 16, 2026
2 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