Repo cleanup: purge leaked creds, dedupe structure, align tech stack - #838
Open
vic-Gray wants to merge 9 commits into
Open
Repo cleanup: purge leaked creds, dedupe structure, align tech stack#838vic-Gray wants to merge 9 commits into
vic-Gray wants to merge 9 commits into
Conversation
…t_cred_input.txt)
… as active direction
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Summary
Security and hygiene cleanup to meet technical-quality standards. All changes are applied as normal commits on top of
mainso this is a clean, reviewable PR.security — remove leaked credential-harvesting files
get_cred.vbs(a Windows Credential Manager GitHub-credential harvester) andgit_cred_input.txtfrom the tree.chore — stop tracking node_modules
node_modules;.gitignorenow ignores them everywhere. Tracked files 2013 → 878.chore — consolidate duplicate project structure
harvest-finance/{backend,frontend}is now canonical at repo root; removed old rootbackend,src,test, and theharvest-financewrapper. Updated docker-compose, CI, CD, security workflows and Makefile paths. Removed junk (redisdump.rdb, jest artifacts, log files,*-pwd.log).chore — remove scratch/PR-note files
.mdfiles; kept real docs (README, CONTRIBUTING, docs/, API_VERSIONING_*, IMPLEMENTATION_SUMMARY).chore — fix Solidity/Stellar mismatch
contracts/→contracts-legacy/with a deprecation README.contracts-soroban/+NOTES.md+ integration tests.ci — stop masking failures
nest build || truemask and unmaskednpm ci/lint/build/testin CI so real failures surface.Verification still required (no toolchain in build sandbox)
npm ci && npm run build && npm run testforbackend/frontendandforge testforcontracts-legacyin a toolchain-equipped environment.cargo testforcontracts-soroban(Soroban SDK) added but not yet compiled here; thei128share-math overflow risk noted inNOTES.mdmust be resolved before mainnet.