Skip to content

Fix/setup flow#28

Merged
stewartshea merged 4 commits intomainfrom
fix/setup-flow
Apr 13, 2026
Merged

Fix/setup flow#28
stewartshea merged 4 commits intomainfrom
fix/setup-flow

Conversation

@stewartshea
Copy link
Copy Markdown
Contributor

No description provided.

- Add .devcontainer/Dockerfile that patches /usr/lib/os-release and
  /etc/os-release to bookworm before docker-in-docker runs (works even
  when GHCR :latest is stale).
- Switch devcontainer.json from image to build using that Dockerfile.
- Mirror the same normalization in the repo root Dockerfile for CI builds.

Made-with: Cursor
Drop .devcontainer/Dockerfile and restore image: in devcontainer.json.
os-release normalization stays in the root Dockerfile (built by Actions).

Made-with: Cursor
- Added .cursor/rules/ to .gitignore to prevent committing generated Cursor rules.
- Updated README to clarify the location of installed skills and the symlink for codecollection.
- Modified Taskfile to create a symlink for codecollection during setup and improved messaging for clarity.
- Removed the on-create.sh script as its functionality is now handled by Taskfile.

These changes enhance the developer experience by streamlining the setup process and providing clearer instructions.
- Updated Dockerfile to create the .ssh directory and set appropriate permissions for SSH access.
- Added a postCreateCommand in devcontainer.json to ensure the .ssh directory and authorized_keys file are created with correct permissions automatically.
- Revised documentation to clarify that SSH permission errors should not occur due to these automated setups.

These changes improve the security and usability of the development environment.
@stewartshea stewartshea merged commit 8b3c6f2 into main Apr 13, 2026
6 checks passed
@stewartshea stewartshea deleted the fix/setup-flow branch April 13, 2026 21:37
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