Repository navigation
Add Built-in process catalog and process show - #1470
Merged
Merged
Conversation
Preserve Process inspection and Resume documentation together, and regenerate the user manual HTML.
This was referenced Oct 27, 2026
Merged
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.
Summary
Adds the Built-in process folder and embedding mechanism, preserves reserved names, and documents inspection and copying. No real Built-in process ships in this ticket;
ticket-runfollows in #1463.Closes #1459
Evidence
process_show_prints_the_custom_file_exactlyfailed becauseprocess showwas unrecognized; compiled-catalog, explicit-list and package-content regressions also failed before their changes.After: Those regressions pass, covering exact newlines/CRLF/empty files, unknown names, compiled Built-in inspection/listing, and reserved-name collisions.
cargo test --test process --test packaging: 128 passed, 1 ignored.cargo test --bin thirdshift processes::catalog::tests: 2 passed.cargo check --bin thirdshift: passed.cargo test --test command_surface: 20 passed, 1 ignored.NEXTEST_TEST_THREADS=8 /usr/bin/time -p scripts/affected-tests issue-1456: 51 site checks passed; the Rust run stopped early on a Console HTTP timeout after 273 passes (437.35s wall, including compilation).cargo test --test console accounts::password_codes::code_expiry_is_enforced_over_http_with_a_delivered_code -- --exact: passed alone in 5.76s.14e912a8: CI passed, 3,278 passed, 74 skipped; 842.776s execution plus 59.07s compilation. Formatting, clippy, site, Console and release checks also passed. The successful CI result is reused instead of repeating the full suite on the same head.Testing seams
thirdshift process show,list, and reserved names.Review coverage
Independent
thirdshift-code-reviewagainstissue-1456: Standards 0, Spec 0. Both reviewers read all 15 changed files throughc1f9e60e. Unread final hunk: the help-text line consolidation insrc/main.rsat14e912a8was added after their single permitted follow-up; the existing compact-help regression passes. Generated-output exclusions: none.Unaddressed findings
Standards
None.
Spec
None.
Security
None.
Security review outcome
No unaddressed introduced findings.
Merge Danger
Door: two-way
Code and documentation can be reverted; there is no stored-data migration.
Blast Radius: Processes
Changes Process discovery and inspection.
showbecomes a reserved command name.Built with codex codex-cli 0.164.0 · gpt-6.1-sol · xhigh