Skip to content

Commit b897bf2

Browse files
committed
refactor: Refactor backend to a rest api and make weak changes to the front to make it works
fix weird behavior after rebase attempt clean after rebase refactor: centralize PAT validation, streamline repo checks & misc cleanup (coderamp-labs#349) * refactor: centralize PAT validation, streamline repo checks & housekeeping * `.venv*` to `.gitignore` * `# type: ignore[attr-defined]` hints in `compat_typing.py` for IDE-agnostic imports * Helpful PAT string in `InvalidGitHubTokenError` for easier debugging * Bump **ruff-pre-commit** hook → `v0.12.1` * CONTRIBUTING: * Require **Python 3.9+** * Recommend signed (`-S`) commits * PAT validation now happens **only** in entry points (`utils.auth.resolve_token` for CLI/lib, `server.process_query` for Web UI) * Unified `_check_github_repo_exists` into `check_repo_exists`, replacing `curl -I` with `curl --silent --location --write-out %{http_code} -o /dev/null` * Broaden `_GITHUB_PAT_PATTERN` * `create_git_auth_header` raises `ValueError` when hostname is missing * Tests updated to expect raw HTTP-code output * Superfluous “token can be set via `GITHUB_TOKEN`” notes in docstrings * `.gitingestignore` & `.terraform` from `DEFAULT_IGNORE_PATTERNS` * Token validation inside `create_git_command` * Obsolete `test_create_git_command_invalid_token` * Adjust `test_clone.py` and `test_git_utils.py` for new status-code handling * Consolidate mocks after token-validation relocation BREAKING CHANGE: `create_git_command` no longer validates GitHub tokens; callers must ensure tokens are valid (via `validate_github_token`) before invoking lower-level git helpers. --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> remove api ingest endpoint test (duplicate) add uv.lock to gitignore
1 parent 5317abf commit b897bf2

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

.gitignore

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,13 +99,14 @@ ipython_config.py
9999
# poetry
100100
# Similar to Pipfile.lock, it is generally recommended to include poetry.lock in version control.
101101
# This is especially recommended for binary packages to ensure reproducibility, and is more
102+
uv.lock
102103
# commonly ignored for libraries.
103104
# https://python-poetry.org/docs/basic-usage/#commit-your-poetrylock-file-to-version-control
104105
#poetry.lock
105106

106107
# pdm
107108
# Similar to Pipfile.lock, it is generally recommended to include pdm.lock in version control.
108-
#pdm.lock
109+
#pdm
109110
# pdm stores project-wide configurations in .pdm.toml, but it is recommended to not include it
110111
# in version control.
111112
# https://pdm.fming.dev/latest/usage/project/#working-with-version-control
@@ -174,6 +175,7 @@ node_modules/
174175
history.txt
175176
cleanup.py
176177
Caddyfile
178+
uv.lock
177179

178180
# ignore default output directory
179181
tmp/*

0 commit comments

Comments
 (0)