feat: add Hub Apps navigation and enforce PR-only main integration - #135
Merged
Conversation
# Conflicts: # tests/e2e/new-issue-project-context.spec.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Require every change to reach
mainthrough a pull request, including release preparation and the post-release version bump. Update contributor and maintainer instructions to remove direct-push exceptions, and replace the next-version push with an idempotent maintenance branch and PR.Maintenance PRs dispatch qualification on their own branch so the required check attaches to the PR commit. An inline bootstrap validates the immutable source and trusted workflow/guard before candidate code runs. Release promotion remains limited to qualified
mainsources. A handoff PR remains explicitly pending until it merges.The live
main-pr-onlyGitHub ruleset is active forrefs/heads/main: no bypass actors, required PR, strict GitHub ActionsQualification summary, no force pushes or deletion. Effective branch rules were independently read back; current administrator bypass isnever.Validation:
634aab5bbCI also fails the architecture ratchet and the log-store cancellation test. These checks remain enforced.Hub Apps
Replace the Hub Showcase tab with an Apps catalog. Opening an App adds an organization-scoped primary-rail entry; its close button appears on hover or keyboard focus. Closing removes the entry and matching saved-view pins while preserving the App, saved views, data, and running processes. The catalog can reopen closed Apps.
Validation: 44 focused unit tests, the real-API Hub Apps E2E, the existing App Builder multiple-tab E2E, UI typecheck, production frontend build, and scoped diff checks passed. Independent rendered acceptance returned PASS and final review accepted the source. Web UI validation covers dark/light themes, constrained layout, active/inactive close, refresh, organization isolation, and saved-view alias cleanup. No new Desktop installer was packaged, and no release or deployment was performed.