From 2d9e5744d398ed594119f57928e27d071a3ef84c Mon Sep 17 00:00:00 2001 From: Prateep07 Date: Thu, 5 Nov 2026 18:10:20 +0530 Subject: [PATCH 1/3] 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/3] 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/3] 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.