Revert community submission intake and outcome reporting changes for additional testing - #4831
Merged
KSchlobohm merged 1 commit intoOct 3, 2026
Conversation
Create space for additional hosted validation before proceeding with or reintroducing the changes from github#4829. Assisted-by: GitHub Copilot App (model: GPT-5.6 Sol Fast (Internal only), autonomous) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The nine-file change consistently reverses #4829 while retaining baseline workflow coverage.
Review effort: Balanced
Findings: None
What changed in this PR
Reverts #4829 to restore the prior community-submission workflow behavior pending additional hosted testing.
Changes:
- Restores exact title-prefix validation for extension, preset, and bundle submissions.
- Removes shared outcome reporting, related documentation, and feature-specific tests.
- Regenerates all three compiled workflow locks.
| File | Description |
|---|---|
.github/workflows/add-community-extension.md |
Restores extension title gating. |
.github/workflows/add-community-extension.lock.yml |
Reverts compiled extension workflow. |
.github/workflows/add-community-preset.md |
Restores preset title gating. |
.github/workflows/add-community-preset.lock.yml |
Reverts compiled preset workflow. |
.github/workflows/add-community-bundle.md |
Restores bundle title gating. |
.github/workflows/add-community-bundle.lock.yml |
Reverts compiled bundle workflow. |
.github/workflows/shared/catalog-submission.md |
Removes shared intake and reporting behavior. |
tests/test_submission_outcomes.py |
Removes tests specific to reverted behavior. |
docs/guides/agentic-sdlc.md |
Removes documentation for reverted behavior. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
mahmoudbauomi-prog
approved these changes
Oct 3, 2026
mahmoudbauomi-prog
approved these changes
Oct 3, 2026
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.
Description
Create space for additional hosted testing before asking upstream to carry the community-submission intake and outcome-reporting changes.
This reverts #4829 and its squash merge commit
de0cbd762e2d0b30f90d3ffcbe2ee9d7167f4edaas an exact nine-file inverse. The contributor wants more confidence in hosted behavior before proceeding with or reintroducing the change, especially around comment ownership and tone, publication-state reconciliation, failure paths, and the long-term test strategy.This is a temporary confidence-building step, not a finding that #4829 is defective or a permanent rejection of its goals. The tradeoff is explicit: extension, preset, and bundle submissions again require the prior exact title prefixes, and title mismatches again stop without an explanatory outcome comment. The shared all-path outcome and fallback reporting is also removed until the additional testing and design work is ready.
The revert intentionally removes the shared prompt, its feature-specific tests, and its documentation because those surfaces exist only for the reverted behavior. Their absence is expected; retained baseline workflow wiring remains covered by the existing workflow tests.
This does not revert or modify #4830, including
tests/specify_cli/workflows/test_catalog_versions.py. It also does not close or otherwise mutate the issues that motivated #4829. No separate fork experimentation is included or required by this upstream revert.Testing
uv run specify --helpuv sync && uv run pytestExact local validation on Windows, using this worktree and Python 3.11.5:
uv sync --extra test.venvand installed test dependencies.gh aw versionv0.88.7.gh aw compile add-community-extension add-community-preset add-community-bundle --no-check-update.gitattributesline was removed..venv\Scripts\python.exe -m pytest tests\test_github_workflows.py -qnpx --yes markdownlint-cli2 "docs/guides/agentic-sdlc.md"git diff --checkandgit diff --cached --checkgit write-treecompared withde0cbd762e2d0b30f90d3ffcbe2ee9d7167f4eda^1^{tree}b1be4e54264194697401b55ec02ad4e2b99c00ff.git apply --cached --check.catalog-submission.md,test_submission_outcomes, andsubmission_outcomesreferencesThe merged #4829 checks and the current
mainchecks were green, including the cross-platform Python matrix, markdownlint, CodeQL, dependency audit, and related repository checks. This revert is not claiming those checks failed; it creates room for additional hosted behavior testing beyond them.No hosted workflow runs were created for this preparation. The full Python suite, CLI help, and sample-project tests were not run because this is the exact inverse of the workflow feature and its feature-only tests, with targeted baseline workflow wiring coverage retained. Hosted failure injection and comment/publication reconciliation scenarios remain the validation work desired before reintroduction.
AI Disclosure
AI disclosure: The contributor directed this revert. GitHub Copilot App, powered by GPT-5.6 Sol, applied the exact revert, compared it with the merged PR patch and file API, regenerated and checked workflow locks, ran local validation, and drafted this pull request body in autonomous/autopilot mode. Reasoning effort was runtime-selected and not explicitly overridden. Publication was explicitly approved by the contributor.