Skip to content

chore(repo): track natively-api as a registered submodule - #505

Merged
evinjohnn merged 1 commit into
mainfrom
chore/natively-api-submodule
Aug 25, 2026
Merged

chore(repo): track natively-api as a registered submodule#505
evinjohnn merged 1 commit into
mainfrom
chore/natively-api-submodule

Conversation

@evinjohnn

Copy link
Copy Markdown
Collaborator

What

natively-api was recorded in the parent index as a gitlink (160000 6217f1f) but had no .gitmodules entry. The parent .gitignore also listed the path twice.

This PR adds the missing submodule registration and drops the two ignore rules.

.gitignore  | 2 --
.gitmodules | 3 +++

Why

Two problems fell out of the orphan gitlink:

  1. Fresh clones got an empty directory. With no URL recorded, git submodule update --init and git clone --recurse-submodules had nothing to fetch, so natively-api/ came down empty.
  2. AI IDEs skipped the API server code. The code is on disk, but IDEs honour .gitignore when indexing, so natively-api / /natively-api in 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 main6217f1f…, exactly the commit the parent pins, so a recursive clone lands on the right code.
  • git submodule status reports both submodules clean and in sync (6217f1f natively-api, ae7b4ba premium).
  • git check-ignore natively-api → no longer ignored.
  • Other natively* ignore rules were left untouched; the bare natively pattern matches only a path component of exactly that name, not natively-api.

Not affected

natively-api keeps its own history and push target, so the Railway deploy from evinjohnn/natively-api@main is unchanged. No secrets are newly tracked — submodule contents are not stored in this repo, and natively-api/.gitignore already covers .env, node_modules, and log.txt.

Note for reviewers

The submodule URL points at evinjohnn/natively-api, a different owner than premium (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

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
@evinjohnn
evinjohnn merged commit 7f23065 into main Aug 25, 2026
4 of 5 checks passed
@greptile-apps

greptile-apps Bot commented Aug 25, 2026

Copy link
Copy Markdown

Greptile Summary

Registers the existing natively-api gitlink as a submodule and removes duplicate ignore rules so recursive clones can populate it and IDEs can index its source.

  • Adds the natively-api path and repository URL to .gitmodules.
  • Removes both parent-level .gitignore entries for that path.
  • Leaves two build-smoke workflow comments obsolete because they still describe the gitlink as lacking a URL.

Confidence Score: 4/5

The PR appears safe to merge, with only stale build-workflow documentation needing cleanup.

The configured submodule matches the existing gitlink and is excluded from packaged artifacts; the remaining issue is that two workflow comments still document the pre-registration failure mode.

Files Needing Attention: .gitmodules and .github/workflows/build-smoke.yml

Important Files Changed

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.

Fix all with Greploop Fix All in Claude Code

Reviews (1): Last reviewed commit: "chore(repo): track natively-api as a reg..." | Re-trigger Greptile

Comment thread .gitmodules
Comment on lines +4 to +6
[submodule "natively-api"]
path = natively-api
url = https://github.com/evinjohnn/natively-api.git

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Submodule registration stales workflow comments

Registering natively-api makes both build-smoke comments claiming that the gitlink has no .gitmodules URL obsolete, giving maintainers an incorrect explanation for the path-limited checkout strategy.

Fix in Claude Code

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
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