Skip to content

chore: proposed implementation plan#140

Closed
shaninja wants to merge 1 commit into
masterfrom
chore/completeness-implementation
Closed

chore: proposed implementation plan#140
shaninja wants to merge 1 commit into
masterfrom
chore/completeness-implementation

Conversation

@shaninja

Copy link
Copy Markdown

No description provided.

@shaninja shaninja self-assigned this Apr 27, 2026
@shaninja shaninja requested a review from lavie April 27, 2026 14:03
- stdin inspect JSON input

2. **image enrichment**
- load image facts when available so image-defined defaults can be recognized and handled consistently

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is image enrichment?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inspecting the image and getting it's defaults. This will allow the library to distinguish “this was explicitly set on docker run” from “this came from the image.” In very specific cases it's also needed. Example from Codex:

there are cases where the absence of a value in container inspect is only meaningful relative to the image default.
Example:
docker run --entrypoint="" image-with-entrypoint sh
The container’s effective entrypoint is empty. If runlike only sees container inspect and emits:
docker run image-with-entrypoint sh
the clone will inherit the image’s default entrypoint, which is wrong. To know that empty entrypoint means “explicitly reset image entrypoint,” runlike needs image facts.

Each dictionary entry covers one canonical option or one tightly related option group and records:

- canonical output form;
- equivalent spellings and aliases;

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might mean that the insect | runlike | run | inspect loop won't work simplistically since there can be equivalent but different options. At least the comparator should be aware of this and forgive insignificant differences (e.g. -i, --interactive).

@shaninja

Copy link
Copy Markdown
Author

closing this PR. all further work will continue on the fork https://github.com/shaninja/runlike

@shaninja shaninja closed this Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants