Daily PlanMarch 4, 2026

[github-actions[bot]]

Executive Summary

Current State: KSail is in excellent shape with very low open item count — only 3 open issues (all blocked on upstream dependencies) and 4 open PRs. Today (March 4) saw a major burst of activity: the enums.go refactor was merged (split into 11 domain-specific files), along with multiple documentation improvements and lint fixes. The repository is clean, well-tested, and actively maintained by the AI team.

Key Metrics:

• 📊 Open Issues: 3 (all blocked on upstream or transient CI)
• 📋 Open PRs: 4 (2 ready today, 1 older feature PR, 1 unknown)
• ✅ Merged Today (March 4): 5+ PRs including major enums.go refactor
• 🤖 Automation: Daily workflows actively running
• 🎯 CI Health: Stable; transient VCluster failures remain a recurring concern
• 🌟 Code Quality: golangci-lint clean after comprehensive gosec/nolintlint pass

Strategic Priorities (March 4, 2026):

1. HIGH: Merge Mermaid diagram fix (fix: escape parentheses in architecture mermaid diagram #2763)
2. HIGH: Merge copilot-instructions.md enum sync docs ([docs] docs: sync copilot-instructions.md with enum split refactor #2762)
3. MEDIUM: Review Omni × Docker CI system test PR (ci: add Talos × Omni cases to system tests #2547)
4. LOW: Monitor and track blocked upstream issues (CI DoctorCI Failure Investigation - Run #6187 (Transient System Test Failure) #2729, chore: remove loft-sh/log fork when upstream updates tablewriter to v1.x #2246)

---

Recent Completions (Since March 3 Plan) ✅

Merged Today (March 4)

✅ PR #2725 (Daily Refactor): Split pkg/apis/cluster/v1alpha1/enums.go (773 lines) into 11 domain-specific files (distribution.go, cni.go, csi.go, loadbalancer.go, certmanager.go, policyengine.go, gitopsengine.go, metricsserver.go, placementgroupstrategy.go, provider.go, enum.go). Included sub-PRs:

• fix: resolve all golangci-lint issues across the repository #2739: Fix all golangci-lint issues after refactor
• fix: address gosec/nolintlint review comments and resolve all golangci-lint issues #2741: Address gosec/nolintlint review comments
• fix: resolve all golangci-lint gosec issues across codebase #2757: Final gosec suppressions cleanup

✅ PR #2752: Reduce bloat in architecture.mdx (534 → 408 lines, 23.6% reduction)

✅ PR #2735: Add architecture and development guide links to copilot-instructions.md

✅ PR #2736 + #2740: Sync Backup & Restore feature docs and update agentic workflows table in CONTRIBUTING.md (consolidated 17 → 13 workflows)

✅ PR #2738: Reduce bloat in loadbalancer.mdx by 23%

Impact: 📁 Improved code organization, 📖 Accurate documentation, ✅ Clean lint slate

---

HIGH PRIORITY: Open PRs Ready to Merge

1. PR #2763: fix: escape parentheses in architecture mermaid diagram

Priority: ⭐⭐⭐⭐⭐ HIGH
Status: Open (Created March 4)
Type: Bug Fix — Documentation

Scope:

• Mermaid parser interprets (Kind) and (K3d) in node labels as shape delimiters (PS token)
• Quotes the node label: Provisioner["Provisioners<br/>Vanilla (Kind), K3s (K3d), Talos, VCluster"]
• Fixes architecture diagram rendering

Action Required:

1. Verify Mermaid renders correctly with quoted labels
2. MERGE — TARGET: March 4, 2026

Impact: 🎨 Architecture diagram renders correctly

---

2. PR #2762: docs: sync copilot-instructions.md with enum split refactor

Priority: ⭐⭐⭐⭐ HIGH
Status: Open (Created March 4)
Type: Documentation Sync

Scope:

• Updates pkg/apis/ package description to reflect the new domain-specific enum files (from PR Daily Refactor - Split enums.go into domain-specific files #2725 refactor)
• Removes stale reference to the now-deleted enums.go monolith
• Ensures Copilot agent has accurate understanding of the new file structure

Action Required:

1. Verify descriptions match the 11 new domain files
2. MERGE — TARGET: March 4, 2026

Impact: 📝 AI agents understand new enum file structure

---

MEDIUM PRIORITY: Older PR Needing Review

3. PR #2547: Add Omni × Docker CI system test support

Priority: ⭐⭐⭐ MEDIUM
Status: Open (Created Feb 25, 3 comments)
Type: Feature / CI Enhancement

Scope:

• Adds Omni provider support to .github/actions/ksail-system-test/action.yaml
• New inputs: omni-service-account-key, omni-endpoint
• Injects Omni endpoint into ksail.yaml via heredoc (non-init) or shell append (init)
• Propagates OMNI_SERVICE_ACCOUNT_KEY to all cluster lifecycle steps

Action Required:

1. Review 3 existing comments for any outstanding concerns
2. Verify secret names match repository secrets (OMNI_SERVICE_ACCOUNT_KEY, OMNI_ENDPOINT)
3. Consider whether Omni CI tests should be continue-on-error: true (like Hetzner)
4. REVIEW and MERGE if approved — TARGET: March 5-6, 2026

Impact: 🚀 Enables automated CI testing for the Omni provider

---

BLOCKED: Open Issues (Monitoring Only)

1. Issue #2729: CI Doctor — Transient System Test Failure (Run #6187)

Status: 🔒 Blocked (transient infrastructure / upstream VCluster)
Labels: blocked

Summary: Transient VCluster/Talos system test failure during PR #2396 merge. PR itself succeeded on re-run. Blocked pending:

• Extended VCluster retry coverage for GHCR denied: denied errors
• Pre-warming VCluster base image in CI

No immediate action needed — monitor for recurrence.

---

2. Issue #2246: Remove loft-sh/log fork when tablewriter v1.x is adopted upstream

Status: 🔒 Blocked (upstream loft-sh/log dependency)
Labels: blocked

Summary: KSail maintains a local fork of loft-sh/log in patches/loft-sh-log/ due to tablewriter API incompatibility (v0.0.5 vs v1.x). Unblocked when upstream submits/merges the tablewriter v1.x PR.

Pending: Submit upstream PR to loft-sh/log with tablewriter v1.x update.

---

3. Third Blocked Issue (VCluster SDK/D-Bus)

Status: 🔒 Blocked (upstream VCluster SDK)

From context: likely tracking VCluster D-Bus race condition in install-standalone.sh (#2261) — needs an upstream issue filed with loft-sh/vcluster to add D-Bus readiness wait before systemctl restart systemd-journald. KSail has a workaround in place (recoverFromDBusError).

---

Suggested New Issues

The following issues are worth creating to track outstanding work not yet captured:

# 1. VCluster: Add image-pull retry for GHCR blob denial in provisioner
gh issue create \
  --title "feat(vcluster): add image-pull retry for transient GHCR blob denial" \
  --body "## Context

Issue #2729 documents GHCR \`denied: denied\` transient failures during VCluster image pull. The current \`createWithRetry\` in \`pkg/svc/provisioner/cluster/vcluster/provisioner.go\` covers \`exit status 22\` (EINVAL) and D-Bus errors but does NOT retry image-pull failures.

## Action Items

- [ ] Add \`fetching blob: denied: denied\` to \`transientCreateErrors()\` patterns
- [ ] Add \`manifest unknown\` and similar GHCR transient errors  
- [ ] Pre-warm \`ghcr.io/loft-sh/kubernetes\` in \`.github/actions/ksail-system-test\` warm-up step
- [ ] Update \`transientCreateErrors()\` unit tests in \`provisioner_test.go\` to cover new patterns
- [ ] Document retry behavior in troubleshooting guide

## References

- Issue #2729: Transient system test failure (GHCR blob denial)
- Issue #2646: Previous GHCR blob access denied failure  
- \`pkg/svc/provisioner/cluster/vcluster/provisioner.go:75-82\`: current transient error patterns" \
  --label enhancement,bug

# 2. Upstream: File issue on loft-sh/vcluster for D-Bus race in install-standalone.sh
gh issue create \
  --title "chore: file upstream issue on loft-sh/vcluster for D-Bus race condition" \
  --body "## Context

KSail implements \`recoverFromDBusError\` in \`pkg/svc/provisioner/cluster/vcluster/provisioner.go\` as a workaround for a race condition in the VCluster SDK's \`install-standalone.sh\`:

- Line 333: \`systemctl restart systemd-journald\` runs before D-Bus is initialized
- Container starts with \`--tmpfs /run\` so \`/run/dbus/system_bus_socket\` doesn't exist immediately
- Combined with \`set -e\`, this aborts the install script on Linux CI runners

## Action Items

- [ ] File upstream issue at https://github.com/loft-sh/vcluster with options:
  - Add D-Bus readiness wait before \`systemctl restart systemd-journald\`
  - Make journald restart non-fatal (\`|| true\`)
  - Add SDK option to skip persistent logging setup
- [ ] Track upstream resolution
- [ ] Remove KSail workaround once upstream fix is released

## References

- \`pkg/svc/provisioner/cluster/vcluster/provisioner.go\`: \`tryDBusRecovery\` helper
- VCluster SDK: github.com/loft-sh/vcluster v0.32.1" \
  --label chore,blocked

# 3. Test: Increase unit test coverage for pkg/cli/ui/chat/ TUI components
gh issue create \
  --title "test: increase unit test coverage for pkg/cli/ui/chat/ TUI components" \
  --body "## Context

The chat TUI package (\`pkg/cli/ui/chat/\`) is under active development (multiple fixes in PRs #2705, #2700, #2702, #2396). Unit test coverage remains low, increasing regression risk as features are added.

## Action Items

- [ ] Audit current test coverage for \`pkg/cli/ui/chat/\`
- [ ] Add tests for ChatMode switching (Agent → Plan → Agent cycle)
- [ ] Add tests for permission handler in Agent mode
- [ ] Add tests for model picker keybinding (\`^O\`)
- [ ] Add tests for reasoning effort keybinding (\`^E\`)
- [ ] Add tests for prompt queuing and steering behavior
- [ ] Add tests for session history keybinding (\`^H\`)
- [ ] Target: 60%+ coverage for core TUI logic
- [ ] Follow \`export_test.go\` pattern for unexported symbols

## References

- \`pkg/cli/ui/chat/model.go\`: ChatMode struct
- PR #2396: Prompt queuing and steering feature" \
  --label testing,enhancement

# 4. Docs: Document enums.go split refactor in architecture guide
gh issue create \
  --title "docs: update architecture guide to reflect enums.go split into domain files" \
  --body "## Context

PR #2725 (Daily Refactor) split \`pkg/apis/cluster/v1alpha1/enums.go\` (773 lines) into 11 domain-specific files. The architecture documentation and \`pkg/apis/\` description in the architecture guide should reflect this new structure.

## Action Items

- [ ] Update \`docs/src/content/docs/architecture.mdx\` API types section
- [ ] Mention domain-specific enum files in the package overview
- [ ] Update any references to \`enums.go\` in the docs site
- [ ] Verify copilot-instructions.md already reflects the change (PR #2762)

## References

- PR #2725: Split enums.go into domain-specific files
- PR #2762: copilot-instructions.md sync (in progress)" \
  --label documentation

---

Summary & Next Steps

Immediate (March 4, 2026)

1. ✅ Merge fix: escape parentheses in architecture mermaid diagram #2763 — Mermaid diagram parentheses fix
2. ✅ Merge [docs] docs: sync copilot-instructions.md with enum split refactor #2762 — copilot-instructions.md enum sync

Short-Term (March 5–6, 2026)

1. Review and merge ci: add Talos × Omni cases to system tests #2547 (Omni CI support)
2. Create suggested issues above (VCluster GHCR retry, D-Bus upstream, chat TUI tests)
3. Trigger daily-code-health to continue test coverage work

Medium-Term (March 7–10, 2026)

1. Submit upstream PR to loft-sh/log for tablewriter v1.x update (unblocks chore: remove loft-sh/log fork when upstream updates tablewriter to v1.x #2246)
2. File upstream issue to loft-sh/vcluster for D-Bus race condition
3. Continue expanding test coverage for pkg/cli/ui/chat/
4. Monitor VCluster system test reliability

Long-Term

1. Monitor VCluster SDK v0.33.x release — check if D-Bus race is fixed
2. Remove patches/loft-sh-log/ fork once upstream merges tablewriter v1.x
3. Consider adding GHCR image-pull retry to VCluster provisioner

---

Workflow Health Summary

Active Agentic Workflows:

• ✅ Daily Refactor (merged major enums.go split today)
• ✅ Daily Docs (bloat reduction, sync docs)
• ✅ Daily Code Health (test coverage improvements)
• ✅ Daily Builder / Daily Plan
• ✅ Daily Workflow Maintenance (workflow action version updates pending via Update workflows - 2026-03-03 #2732)

CI/CD Health:

• ✅ Unit tests passing
• ✅ golangci-lint clean (comprehensive gosec fixes merged)
• ✅ System tests stable
• ⚠️ Transient VCluster/GHCR failures continue to occur (tracking in CI DoctorCI Failure Investigation - Run #6187 (Transient System Test Failure) #2729)

---

Team Velocity

Last 7 Days:

• 🚀 ~30 PRs merged (sustained high velocity)
• 🔧 Major refactor: enums.go split into 11 domain files
• 📝 Documentation quality: Multiple bloat-reduction passes
• ✅ Lint quality: Comprehensive gosec suppression cleanup
• 🛡️ CI reliability: Ongoing VCluster retry improvements

Current Sprint Focus:

• Merging pending PRs (fix: escape parentheses in architecture mermaid diagram #2763, [docs] docs: sync copilot-instructions.md with enum split refactor #2762, ci: add Talos × Omni cases to system tests #2547)
• Expanding test coverage for chat TUI
• Upstream engagement for loft-sh/log and loft-sh/vcluster

---

Generated by Daily Plan workflow on March 4, 2026
Previous plan: Daily Plan - March 3, 2026

AI generated by Daily Plan

• expires on Mar 11, 2026, 10:16 AM UTC