Skip to content

Fix cat id_rsa easter egg for missing files - #140

Open
r-sandeep wants to merge 1 commit into
rootvc:mainfrom
r-sandeep:asd/bugfix/cat-id-rsa-existence-guard
Open

Fix cat id_rsa easter egg for missing files#140
r-sandeep wants to merge 1 commit into
rootvc:mainfrom
r-sandeep:asd/bugfix/cat-id-rsa-existence-guard

Conversation

@r-sandeep

Copy link
Copy Markdown
Contributor

Summary

  • move the id_rsa easter-egg trigger into cat's successful file-existence branch
  • preserve the existing file-content and missing-file output behavior
  • add focused regression coverage for both present and absent id_rsa listings

Defect

Previously, cat id_rsa called SpawnRickRollPointers() even when id_rsa was not listed in the current directory. This caused the command to print No such file: id_rsa and still start the easter egg.

Fix

SpawnRickRollPointers() now runs only after id_rsa is found and its contents are printed. Missing files continue to emit the unchanged No such file: <filename> message without triggering pointer output.

Validation

  • npm test — passed (121 tests)
  • npm run build — passed
  • independent verification and adversarial review passed with no unresolved findings

Scope

Only the requested files changed:

  • config/commands.js
  • tests/commands.test.js

Approvals

All deterministic policy decisions allow this change, and the work packet has no open approvals.

ASD references

  • Workflow run: workflow_run_a4f3bece706f4bb0a12a960d84f0c27e
  • Task/work packet: task_0b65e8f3857d4abba385282c4c9aa5c3

…rigger, added focused present/absent regression coverage, fixed the test harness setup, and passed the required
@r-sandeep
r-sandeep marked this pull request as ready for review August 26, 2026 01:27
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.

1 participant