From 2d9e5744d398ed594119f57928e27d071a3ef84c Mon Sep 17 00:00:00 2001 From: Prateep07 Date: Thu, 5 Nov 2026 18:10:20 +0530 Subject: [PATCH 1/4] docs: update file length issue breakdown --- docs/investigations/file-length-limit.md | 30 +++++++++++++----------- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/docs/investigations/file-length-limit.md b/docs/investigations/file-length-limit.md index 2beb7da..70dc7ad 100644 --- a/docs/investigations/file-length-limit.md +++ b/docs/investigations/file-length-limit.md @@ -148,17 +148,19 @@ Implementation issue scopes: new Python files; block baseline growth; reduce baselines as files shrink; add the CI step; document the scope and update/create `CONTRIBUTING.md`. Out of scope: splitting files or changing line counting to logical SLOC. -2. [**Split core application and CLI modules behind compatibility facades**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/112) (**filed**). Use the boundaries above, preserve - `nitrostack.core.app` imports and the `nitrostack.cli.main:main` entry point, - and add/update focused tests for the public imports and generated CLI output. - Out of scope: unrelated behavior changes or broad code cleanup. -3. [**Split remaining over-limit Python files by responsibility**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/113) (**filed**). Batch related - implementation files with their tests (HTTP/widgets/auth/packing), then split - the large test modules and flight-booking example by topic. Start `app.py` - and `http.py` after PR #37 merges, and `cli/main.py` after PR #16 merges; then - remeasure from the latest `develop`. Recheck all open PRs before assigning - work. Out of scope: unrelated changes and line-count-only reshuffling. - -GitHub issues #111, #112, and #113 are open. PRs #37 and #16 remain open and -touch planned split targets; wait for them to merge, then remeasure `develop` -before starting the overlapping splits. +2. [**Split core application and CLI modules behind compatibility facades**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/112) (**open, umbrella**). Its two implementation scopes are tracked as standalone issues: + [CLI `main.py` (#119)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/119) + and [core `app.py` (#120)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/120). + Preserve the CLI entry point and `nitrostack.core.app` compatibility imports. +3. [**Split remaining over-limit Python files by responsibility**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/113) (**open, umbrella**). The implementation scopes are tracked as standalone issues: + [widget route templates and views (#121)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/121), + [HTTP transport and OAuth (#122)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/122), + and [CLI packaging (#123)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/123). + The standalone issues exclude tests and example-related files. + +Before starting an overlapping split, refresh open-PR changed-file lists and +remeasure from the latest `develop`. PRs #37 and #16 were previously identified +as overlapping the core app, HTTP transport, and CLI main targets; confirm their +current status before work begins. The large test modules and flight-booking +example listed above are not included in issues #119–#123 and need separate +scoping if they are to be split. From 0841e8665fb7198f221bd0ab41577f06ec80bfe8 Mon Sep 17 00:00:00 2001 From: Prateep07 Date: Thu, 5 Nov 2026 18:21:22 +0530 Subject: [PATCH 2/4] docs: separate OAuth and transport issue scopes --- docs/investigations/file-length-limit.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/docs/investigations/file-length-limit.md b/docs/investigations/file-length-limit.md index 70dc7ad..c1f0a9f 100644 --- a/docs/investigations/file-length-limit.md +++ b/docs/investigations/file-length-limit.md @@ -154,8 +154,9 @@ Implementation issue scopes: Preserve the CLI entry point and `nitrostack.core.app` compatibility imports. 3. [**Split remaining over-limit Python files by responsibility**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/113) (**open, umbrella**). The implementation scopes are tracked as standalone issues: [widget route templates and views (#121)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/121), - [HTTP transport and OAuth (#122)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/122), - and [CLI packaging (#123)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/123). + [OAuth (#122)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/122), + [CLI packaging (#123)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/123), + and [HTTP transport (#125)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/125). The standalone issues exclude tests and example-related files. Before starting an overlapping split, refresh open-PR changed-file lists and From cecb0f13e19c5c5f95263b721fccc22964f8be4f Mon Sep 17 00:00:00 2001 From: Prateep07 Date: Thu, 5 Nov 2026 18:24:01 +0530 Subject: [PATCH 3/4] docs: include flight example in CLI split scope --- docs/investigations/file-length-limit.md | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/docs/investigations/file-length-limit.md b/docs/investigations/file-length-limit.md index c1f0a9f..c6aaea2 100644 --- a/docs/investigations/file-length-limit.md +++ b/docs/investigations/file-length-limit.md @@ -149,7 +149,7 @@ Implementation issue scopes: add the CI step; document the scope and update/create `CONTRIBUTING.md`. Out of scope: splitting files or changing line counting to logical SLOC. 2. [**Split core application and CLI modules behind compatibility facades**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/112) (**open, umbrella**). Its two implementation scopes are tracked as standalone issues: - [CLI `main.py` (#119)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/119) + [CLI `main.py` and flight-booking example (#119)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/119) and [core `app.py` (#120)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/120). Preserve the CLI entry point and `nitrostack.core.app` compatibility imports. 3. [**Split remaining over-limit Python files by responsibility**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/113) (**open, umbrella**). The implementation scopes are tracked as standalone issues: @@ -157,11 +157,11 @@ Implementation issue scopes: [OAuth (#122)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/122), [CLI packaging (#123)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/123), and [HTTP transport (#125)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/125). - The standalone issues exclude tests and example-related files. + The standalone issues exclude tests. The flight-booking example is included + in #119; other example-related files remain out of scope. Before starting an overlapping split, refresh open-PR changed-file lists and remeasure from the latest `develop`. PRs #37 and #16 were previously identified as overlapping the core app, HTTP transport, and CLI main targets; confirm their -current status before work begins. The large test modules and flight-booking -example listed above are not included in issues #119–#123 and need separate -scoping if they are to be split. +current status before work begins. The large test modules listed above are not +included in issues #119–#125 and need separate scoping if they are to be split. From 40cc0cf7ecaedcf86cb5c0a1057b05c5e8b901b2 Mon Sep 17 00:00:00 2001 From: Prateep07 Date: Fri, 6 Nov 2026 19:41:05 +0530 Subject: [PATCH 4/4] docs: refresh file length issue statuses --- docs/investigations/file-length-limit.md | 28 +++++++++++------------- 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/docs/investigations/file-length-limit.md b/docs/investigations/file-length-limit.md index c6aaea2..0c1b1ed 100644 --- a/docs/investigations/file-length-limit.md +++ b/docs/investigations/file-length-limit.md @@ -40,18 +40,18 @@ Actions workflow runs the test suite and coverage only; it does not run Ruff. At the measured commit there was no `docs/` tree or CONTRIBUTING guide; this investigation adds the first `docs/` file, and a CONTRIBUTING guide still does not exist. The open PR list and changed files were checked on 2026-11-04 using -GitHub's public API. Two open PRs touch candidates for the proposed splits: +GitHub's public API. Two PRs that were open at the time of this snapshot touched +candidates for the proposed splits: -| Open PR | Changed over-limit files | Diff size in those files | Scheduling impact | +| PR (status now) | Changed over-limit files | Diff size in those files | Follow-up | | --- | --- | ---: | --- | -| [#37, wire alignment](https://github.com/nitrocloudofficial/nitrostack-python-sdk/pull/37) | `nitrostack/core/app.py`, `nitrostack/transports/http.py` | `app.py`: +12/-32; `http.py`: +22/-6 | Let this PR land before splitting either file, then remeasure. | -| [#16, skip init prompts for non-TTY](https://github.com/nitrocloudofficial/nitrostack-python-sdk/pull/16) | `nitrostack/cli/main.py` | +10/-3 | Let this PR land before splitting the CLI module, then remeasure. | +| [#37, wire alignment (merged)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/pull/37) | `nitrostack/core/app.py`, `nitrostack/transports/http.py` | `app.py`: +12/-32; `http.py`: +22/-6 | Recheck line counts against current `develop` before implementation. | +| [#16, skip init prompts for non-TTY (merged)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/pull/16) | `nitrostack/cli/main.py` | +10/-3 | Recheck line counts against current `develop` before implementation. | The other open PRs at the time of the check (#110, #104, #98, and #58) did not -change any of the 14 files over 500 lines. The overlaps found are small by line -count, but they are direct edits to planned split targets, so sequence those -splits after the PRs merge. This check is a snapshot; repeat it when filing and -scheduling implementation work. +change any of the 14 files over 500 lines. The overlaps found were small by line +count but directly touched planned split targets. This check is a snapshot; +remeasure before scheduling implementation work. ## Decision @@ -148,11 +148,11 @@ Implementation issue scopes: new Python files; block baseline growth; reduce baselines as files shrink; add the CI step; document the scope and update/create `CONTRIBUTING.md`. Out of scope: splitting files or changing line counting to logical SLOC. -2. [**Split core application and CLI modules behind compatibility facades**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/112) (**open, umbrella**). Its two implementation scopes are tracked as standalone issues: +2. [**Split core application and CLI modules behind compatibility facades**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/112) (**closed**). Its implementation scopes are tracked as standalone issues: [CLI `main.py` and flight-booking example (#119)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/119) and [core `app.py` (#120)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/120). Preserve the CLI entry point and `nitrostack.core.app` compatibility imports. -3. [**Split remaining over-limit Python files by responsibility**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/113) (**open, umbrella**). The implementation scopes are tracked as standalone issues: +3. [**Split remaining over-limit Python files by responsibility**](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/113) (**closed**). Its implementation scopes are tracked as standalone issues: [widget route templates and views (#121)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/121), [OAuth (#122)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/122), [CLI packaging (#123)](https://github.com/nitrocloudofficial/nitrostack-python-sdk/issues/123), @@ -160,8 +160,6 @@ Implementation issue scopes: The standalone issues exclude tests. The flight-booking example is included in #119; other example-related files remain out of scope. -Before starting an overlapping split, refresh open-PR changed-file lists and -remeasure from the latest `develop`. PRs #37 and #16 were previously identified -as overlapping the core app, HTTP transport, and CLI main targets; confirm their -current status before work begins. The large test modules listed above are not -included in issues #119–#125 and need separate scoping if they are to be split. +Before starting a split, remeasure from the latest `develop`. The large test +modules listed above are not included in issues #119–#125 and need separate +scoping if they are to be split.