Skip archived repositories using installation listing metadata

[decyjphr]

Archived repositories should not consume per-repository API calls or receive child-plugin writes unless configuration explicitly requests unarchiving. This selectively adapts #1065 to the current yadhav/fix-recent-issues implementation without importing unrelated source history.

Approach

• Pass the installation listing's archived flag separately from the { owner, repo } reference, preventing observed state from leaking into PATCH payloads and accidentally rearchiving a repository.
• Resolve organization, suborganization, repository overrides and plugin-disable settings before skipping. Restore the missing live-state fallback for unknown-state callers and protect child-plugin-only configurations.
• Preserve bounded batching, error aggregation, exclusions, and archive/unarchive ordering. Add real Octokit request-level regressions, a focused smoke phase, and updated documentation.
• Incorporate the latest base's native Probot integration suite and adapt its fixtures to the archive-state lookup, with an archived collaborator regression.

Source audit

Both source commits require adaptation rather than a source-branch merge; source-only ancestry contains no additional merges or unrelated commits.

Source commit	Decision	Adapted commit
e8d49dc90a9b70a9ed148f68a55214979131c4c0	ADAPT	7a064fd
3d5c064f253f30d3b40cfbff5dabbaeb041317eb	ADAPT	7a064fd

Base 3652444 was merged in 302ef36 after the initial adaptation on 55b52df.

Validation

On Node 22.12.0:

• 915 unit tests pass; 12 remain skipped.
• 26 archived-repository transport tests pass, including the maintained smoke phase's complete offline lifecycle. Before the production fix, 16 of the original 25 regression cases failed.
• The merged native integration suite passes all 11 tests, including CI JUnit reporting.
• Existing pagination transport coverage passed all 28 tests on the initial adaptation. No new lint diagnostics; merge-resolution files and whitespace checks pass.

Authorized live smoke on 7a064fd ran only setup -> phase 25 -> teardown: 32 assertions passed, zero failures, no fatal error. It verified actual listing-to-processing metadata propagation, zero fixture requests for archived skips, unknown-state fallbacks, unarchive NOP/apply, safe PATCH payloads, and convergence. Independent before/after snapshots confirmed original files, policies, properties, and foreign refs were unchanged; owned fixtures and processes were removed, the port was freed, and the private environment copy was restored.

The live smoke used controlled single-test-organization enumeration and fixture-only in-memory configuration. It does not establish webhook delivery, multi-organization behavior, or combined integration with #1070. The subsequent base merge was validated locally, not with another live smoke run.

[Copilot]

Copilot review overview

🟡 Changes recommended

Unknown-state syncs can still write to archived repositories when the archive plugin is disabled, and schema documentation remains inconsistent.

Review effort: Balanced
Findings: 1 · 1

Open (2)

• Fetch archive state when reconciliation is disabled · New
• Remove misleading archived defaults from published schemas · New

What changed in this PR

Optimizes archived-repository synchronization by using installation-list metadata while preserving explicit unarchive behavior and fallback checks.

Changes:

• Threads archive metadata through repository processing.
• Adds regression, integration, and smoke coverage.
• Documents archive handling and focused test commands.

File	Description
lib/​settings.js	Adds metadata-aware archive guards and fallbacks.
package.json	Adds archived transport test command.
README.md	Documents transport tests and smoke phase.
smoke-test.js	Adds archived-repository lifecycle phase.
docs/​github-settings/​1. repository-settings.md	Documents archive-state behavior.
test/​unit/​lib/​settings.test.js	Updates method-call expectations.
test/​unit/​lib/​settings-batching.test.js	Tests metadata propagation and batching.
test/​integration/​transport/​archived-repositories.test.js	Adds request-level archive regressions.
test/​integration/​transport/​variables.test.js	Updates imports after relocation.
test/​integration/​transport/​teams.test.js	Updates imports after relocation.
test/​integration/​triggers/​repository-created.test.js	Mocks archive-state lookup.
test/​integration/​triggers/​repository-edited.test.js	Mocks archive-state lookup.
test/​integration/​plugins/​teams.test.js	Supports archive-state lookup.
test/​integration/​plugins/​collaborators.test.js	Tests archived collaborator skipping.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.