Skip to content

Commit 210d8fb

Browse files
committed
fix(container): refresh runtime codex resume hint
1 parent 1e24564 commit 210d8fb

2 files changed

Lines changed: 10 additions & 4 deletions

File tree

packages/container/src/core/templates-entrypoint/codex-resume-hint.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ const escapeForDoubleQuotes = (value: string): string => {
1212

1313
const entrypointCodexResumeHintTemplate = `# Ensure codex resume hint is shown for interactive shells
1414
CODEX_HINT_PATH="/etc/profile.d/zz-codex-resume.sh"
15-
if [[ ! -s "$CODEX_HINT_PATH" ]]; then
16-
cat <<'EOF' > "$CODEX_HINT_PATH"
15+
cat <<'EOF' > "$CODEX_HINT_PATH"
1716
docker_git_workspace_context_line() {
1817
REPO_REF_VALUE="\${REPO_REF:-__REPO_REF_DEFAULT__}"
1918
REPO_URL_VALUE="\${REPO_URL:-__REPO_URL_DEFAULT__}"
@@ -83,8 +82,7 @@ if [ -n "$ZSH_VERSION" ]; then
8382
fi
8483
fi
8584
EOF
86-
chmod 0644 "$CODEX_HINT_PATH"
87-
fi
85+
chmod 0644 "$CODEX_HINT_PATH"
8886
if ! grep -q "zz-codex-resume.sh" /etc/bash.bashrc 2>/dev/null; then
8987
printf "%s\\n" "if [ -f /etc/profile.d/zz-codex-resume.sh ]; then . /etc/profile.d/zz-codex-resume.sh; fi" >> /etc/bash.bashrc
9088
fi

packages/container/tests/core/templates.test.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -448,6 +448,14 @@ describe("renderEntrypoint clone cache", () => {
448448
expect(entrypoint).not.toContain("'+refs/merge-requests/*:refs/merge-requests/*'")
449449
})
450450

451+
it("rewrites the managed Codex resume hint at container startup", () => {
452+
const entrypoint = renderEntrypoint(makeTemplateConfig())
453+
454+
expect(entrypoint).toContain('cat <<\'EOF\' > "$CODEX_HINT_PATH"')
455+
expect(entrypoint).toContain('chmod 0644 "$CODEX_HINT_PATH"')
456+
expect(entrypoint).not.toContain('if [[ ! -s "$CODEX_HINT_PATH" ]]; then')
457+
})
458+
451459
it("preserves branch/tag-only clone-cache refspecs for generated configs", () => {
452460
fc.assert(
453461
fc.property(generatedTemplateConfigArbitrary, (config) => {

0 commit comments

Comments
 (0)