release: publish Socket 9.32.0 - #179
Conversation
Why: Publish the repository workflow and Git-baseline improvements as Socket's next minor release. Verification: uv run pytest plugins/repository-skills uv run scripts/validate_hermes_compatibility.py uv run scripts/validate_socket_metadata.py uv run mypy
|
Important Review skippedToo many files! This PR contains 123 files, which is 23 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (11)
📒 Files selected for processing (123)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 103d6f65b8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "profiles/xcode-workspace/repo-maintenance/validations/40-xcode-workspace-layout.sh", | ||
| "scripts/repo-maintenance/validations/40-xcode-workspace-layout.sh", | ||
| ), |
There was a problem hiding this comment.
When a repository previously installed xcode-workspace and is later refreshed as xcode-app, this workspace-only validator remains in the shared Scripts/repo-maintenance/validations/ directory because refresh preserves files absent from the new profile's target pairs. The generic dispatcher still executes every validation script, so the selected xcode-app profile fails CI as soon as the old workspace layout is removed. Remove obsolete managed overlays during profile transitions or make this validator exit unless the active profile is xcode-workspace.
Useful? React with 👍 / 👎.
| if ! create_github_release; then | ||
| emit_continuation_packet "$pr_number" "$branch_name" "awaiting-github-release-visibility" | ||
| return 0 |
There was a problem hiding this comment.
When GitHub release creation or its immediate visibility check fails here, the flow has already merged the PR and switched to main, but this continuation packet resumes with inspect; inspect then searches for a PR headed by the current main branch, while advance rejects running from main. Thus an eventual-consistency or transient gh release create failure can leave a pushed tag without a verified GitHub release or branch cleanup and no usable scripted resume path. Emit a post-merge operation that can resume from the base branch and verify/create the release idempotently.
AGENTS.md reference: AGENTS.md:L33-L33
Useful? React with 👍 / 👎.
| PYTHON_SOURCE_ROOT = REPO_ROOT / "plugins" / "python-skills" / "skills" | ||
| JVM_SOURCE_ROOT = REPO_ROOT / "plugins" / "server-side-jvm" / "skills" | ||
| CLOUD_DEPLOYMENT_SOURCE_ROOT = REPO_ROOT / "plugins" / "cloud-deployment-skills" / "skills" | ||
| REPOSITORY_SOURCE_ROOT = REPO_ROOT / "plugins" / "repository-skills" / "skills" |
There was a problem hiding this comment.
This new mapping makes plugins/repository-skills/skills the input for repository-skill exports even though the repository declares top-level skills/ to be the authored source and child plugin trees to be packaging. Because write_export() deletes and recreates the top-level export, a future canonical edit under skills/maintain-project-repo or the other repository skills will either fail --check or be overwritten from the packaging copy. Reverse this source direction so export and validation preserve the documented source of truth.
AGENTS.md reference: AGENTS.md:L53-L55
Useful? React with 👍 / 👎.
1 participant
Summary
Verification
platform darwin -- Python 3.14.3, pytest-9.0.3, pluggy-1.6.0
rootdir: /Users/galew/Workspace/gaelic-ghost/socket-wt-repo-version-control-skills/plugins/repository-skills
configfile: pyproject.toml
collected 30 items
plugins/repository-skills/skills/maintain-github-repository/tests/test_maintain_github_repository.py . [ 3%]
.. [ 10%]
plugins/repository-skills/skills/maintain-project-repo/tests/test_maintain_project_repo_workflow.py . [ 13%]
...................s... [ 90%]
plugins/repository-skills/skills/repository-operations-workflow/tests/test_repository_operations_workflows.py . [ 93%]
.. [100%]
======================== 29 passed, 1 skipped in 1.12s =========================
Validating marketplace entries...
Socket marketplace validation passed.