Skip to content

Release: 2 features, 1 fix, 5 chores → Main#8072

Merged
h0lybyte merged 8 commits intomainfrom
dev
Mar 16, 2026
Merged

Release: 2 features, 1 fix, 5 chores → Main#8072
h0lybyte merged 8 commits intomainfrom
dev

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Mar 16, 2026 •
edited
Loading

Release: Dev → Main

8 atomic commits ready for main

Features

Bug Fixes

Chores

This PR is automatically maintained by CI — KBVE Studio

@h0lybyte
fix(vector): use .kubernetes.* field paths and bump to v0.54.0 (#8071)
a0e4323 
kubernetes_logs source nests metadata under .kubernetes.*, not at
top level. All events were falling to router._unmatched because
.container_name was null. Also:
- Bump Vector 0.49.0 → 0.54.0
- Add router._unmatched to ch_normalize inputs
- Add edge-functions container name to functions route
@github-project-automation github-project-automation bot moved this to Review in KBVE Mar 16, 2026
@h0lybyte
feat(isometric): guest auth and auto-reconnect on tab resume (#8073)
41f64a9 
- Server accepts empty JWT as guest session (guest_0, guest_1, …) even
  when SUPABASE_JWT_SECRET is set, so unauthenticated players can join
- JS GoOnlineButton tracks wantOnline state and listens for
  visibilitychange — auto-reconnects when tab resumes and connection
  was lost
- Shared doConnect() with concurrency guard prevents duplicate attempts
@github-actions
Copy link
Contributor Author

github-actions bot commented Mar 16, 2026 •
edited
Loading

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions github-actions bot changed the title Release: 1 fix → Main Release: 1 feature, 1 fix → Main Mar 16, 2026
@h0lybyte
feat(discordsh): add per-player Inventory<ProtoItemKind> sync to Comb…
c19a078 
…atWorld (#8010) (#8070)

Sync player inventories into bevy_inventory Inventory<ProtoItemKind>
during CombatWorld creation and write them back on sync_out. Add
consume_item, add_item, and item_count helpers that use the ECS
inventory's stacking/removal logic instead of manual Vec manipulation.
@github-actions github-actions bot changed the title Release: 1 feature, 1 fix → Main Release: 2 features, 1 fix → Main Mar 16, 2026
@github-actions github-actions bot changed the title Release: 2 features, 1 fix → Main Release: 2 features, 1 fix, 1 chore → Main Mar 16, 2026
@github-actions
chore(astro-kbve): weekly nx report and graph 2026-03-16 (#8074)
ea22358 
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot changed the title Release: 2 features, 1 fix, 1 chore → Main Release: 2 features, 1 fix, 2 chores → Main Mar 16, 2026
@github-actions github-actions bot changed the title Release: 2 features, 1 fix, 2 chores → Main Release: 2 features, 1 fix, 3 chores → Main Mar 16, 2026
@h0lybyte
chore: remove pirate17 gamejam application (#8075)
303c12d 
Backup saved to ~/Documents/pirate17.zip before removal.
@github-actions github-actions bot changed the title Release: 2 features, 1 fix, 3 chores → Main Release: 2 features, 1 fix, 4 chores → Main Mar 16, 2026
github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 16, 2026
View reviewed changes
apps/kbve/astro-kbve/public/isometric/assets/index.js
if (Q === 1) U.children = H;
else if (1 < Q) {
for (var te = Array(Q), He = 0; He < Q; He++)
for (Z in (D.key !== void 0 && (V = '' + D.key), D))

Check warning

Code scanning / CodeQL

Variable not declared before use Warning

Variable 'Z' is used before its
declaration
Loading
.

Copilot Autofix

AI 2 days ago

In general, to fix “variable used before declaration” issues with var, move the var declaration so it appears before any usage, or add an early declaration and turn the later var into a simple assignment. This maintains behavior because var is function-scoped and hoisted, but clarifies intent and avoids reliance on hoisting for readability and tooling.

Here, Z is first used as the loop variable in for (Z in (D.key !== void 0 && (V = '' + D.key), D)) on line 513, but is declared later on line 520 as var Z = arguments.length - 2;. The minimal, behavior-preserving fix is:

  • Introduce a standalone var Z; declaration just before the for (Z in ...) loop.
  • Change the later var Z = arguments.length - 2; to Z = arguments.length - 2; so we don’t redeclare it.

This way, Z is already declared (and initialized to undefined, same as hoisting) when the for loop first uses it, and its later numeric value assignment remains unchanged logically.

Concretely in apps/kbve/astro-kbve/public/isometric/assets/index.js inside react_production.cloneElement = function (m, D, H) { ... }:

  • Right before line 513’s for (Z in ...), add var Z;.
  • Replace the var Z = arguments.length - 2; line at 520 with Z = arguments.length - 2;.

No imports or additional helpers are needed.

Suggested changeset 1
apps/kbve/astro-kbve/public/isometric/assets/index.js

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/apps/kbve/astro-kbve/public/isometric/assets/index.js b/apps/kbve/astro-kbve/public/isometric/assets/index.js
--- a/apps/kbve/astro-kbve/public/isometric/assets/index.js
+++ b/apps/kbve/astro-kbve/public/isometric/assets/index.js
@@ -509,7 +509,8 @@
 				);
 			var U = ze({}, m.props),
 				V = m.key;
-			if (D != null)
+			if (D != null) {
+				var Z;
 				for (Z in (D.key !== void 0 && (V = '' + D.key), D))
 					!Be.call(D, Z) ||
 						Z === 'key' ||
@@ -517,7 +518,8 @@
 						Z === '__source' ||
 						(Z === 'ref' && D.ref === void 0) ||
 						(U[Z] = D[Z]);
-			var Z = arguments.length - 2;
+			}
+			Z = arguments.length - 2;
 			if (Z === 1) U.children = H;
 			else if (1 < Z) {
 				for (var te = Array(Z), He = 0; He < Z; He++)
EOF
@@ -509,7 +509,8 @@
);
var U = ze({}, m.props),
V = m.key;
if (D != null)
if (D != null) {
var Z;
for (Z in (D.key !== void 0 && (V = '' + D.key), D))
!Be.call(D, Z) ||
Z === 'key' ||
@@ -517,7 +518,8 @@
Z === '__source' ||
(Z === 'ref' && D.ref === void 0) ||
(U[Z] = D[Z]);
var Z = arguments.length - 2;
}
Z = arguments.length - 2;
if (Z === 1) U.children = H;
else if (1 < Z) {
for (var te = Array(Z), He = 0; He < Z; He++)
Copilot is powered by AI and may make mistakes. Always verify output.
Unable to commit as this autofix suggestion is now outdated
@h0lybyte
chore(kbve): remove stale sitemaps and replace migration plan with RE…
f5ccb78 
…ADME (#8076)

Remove sitemap-0.xml and sitemap-index.xml (build artifacts that should
not be checked in). Replace completed KBVE_PLAN.md with a concise
README.md documenting the service layout.
@github-actions github-actions bot changed the title Release: 2 features, 1 fix, 4 chores → Main Release: 2 features, 1 fix, 5 chores → Main Mar 16, 2026
@h0lybyte h0lybyte merged commit 145c719 into main Mar 16, 2026
16 checks passed
@github-project-automation github-project-automation bot moved this from Review to Done in KBVE Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

auto-pr dev→main

Projects

KBVE
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@h0lybyte