chore(repo): track natively-api as a registered submodule - #505
Merged
Conversation
natively-api was recorded in the index as a gitlink but had no .gitmodules entry, so `git submodule update --init` and fresh recursive clones left the directory empty. The parent .gitignore also listed the path, which kept AI IDEs from indexing the API server code even though it is present on disk. Register the submodule with its real URL and drop the two ignore rules. natively-api keeps its own history and push target, so the Railway deploy from evinjohnn/natively-api@main is unaffected. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VsosiiexLej7S5JLUYmqhc
|
| Filename | Overview |
|---|---|
| .gitmodules | Correctly registers the existing gitlink, but makes duplicated build-smoke comments about the missing registration obsolete. |
| .gitignore | Removes redundant ignore coverage for the newly registered submodule without affecting parent Git status or release packaging. |
Reviews (1): Last reviewed commit: "chore(repo): track natively-api as a reg..." | Re-trigger Greptile
Comment on lines
+4
to
+6
| [submodule "natively-api"] | ||
| path = natively-api | ||
| url = https://github.com/evinjohnn/natively-api.git |
Abitesh
pushed a commit
to Abitesh/Sneak-Peek
that referenced
this pull request
Aug 30, 2026
…nt/chore/natively-api-submodule chore(repo): track natively-api as a registered submodule
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.
What
natively-apiwas recorded in the parent index as a gitlink (160000 6217f1f) but had no.gitmodulesentry. The parent.gitignorealso listed the path twice.This PR adds the missing submodule registration and drops the two ignore rules.
Why
Two problems fell out of the orphan gitlink:
git submodule update --initandgit clone --recurse-submoduleshad nothing to fetch, sonatively-api/came down empty..gitignorewhen indexing, sonatively-api//natively-apiin the parent ignore file kept the server source out of the index — which is the actual motivation here: making the API code visible alongside the app for compatibility checks.Verification
git ls-remote https://github.com/evinjohnn/natively-api.git main→6217f1f…, exactly the commit the parent pins, so a recursive clone lands on the right code.git submodule statusreports both submodules clean and in sync (6217f1f natively-api,ae7b4ba premium).git check-ignore natively-api→ no longer ignored.natively*ignore rules were left untouched; the barenativelypattern matches only a path component of exactly that name, notnatively-api.Not affected
natively-apikeeps its own history and push target, so the Railway deploy fromevinjohnn/natively-api@mainis unchanged. No secrets are newly tracked — submodule contents are not stored in this repo, andnatively-api/.gitignorealready covers.env,node_modules, andlog.txt.Note for reviewers
The submodule URL points at
evinjohnn/natively-api, a different owner thanpremium(Natively-AI-assistant/natively-premium). That reflects where the repo actually lives today — worth confirming it is the intended long-term home before merge.🤖 Generated with Claude Code
https://claude.ai/code/session_01VsosiiexLej7S5JLUYmqhc