ERROR Argument `SimpleNamespace` is not assignable to parameter

[asukaminato0721]

+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:138:42
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app_model` with type `App` in function `services.app_dsl_service.AppDslService.check_dependencies` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:579:55
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app_model` with type `App` in function `services.app_dsl_service.AppDslService.check_dependencies` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:607:55
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app` with type `App | None` in function `services.app_dsl_service.AppDslService._create_or_update_app` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:664:17
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app` with type `App | None` in function `services.app_dsl_service.AppDslService._create_or_update_app` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:738:21
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app` with type `App | None` in function `services.app_dsl_service.AppDslService._create_or_update_app` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:755:21
+ERROR Attribute `app_model_config` of class `App` is a read-only property and cannot be set [read-only]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:772:9
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app` with type `App | None` in function `services.app_dsl_service.AppDslService._create_or_update_app` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:793:21
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app_model` with type `App` in function `services.app_dsl_service.AppDslService.export_dsl` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:833:34
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app_model` with type `App` in function `services.app_dsl_service.AppDslService.export_dsl` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:847:34
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app_model` with type `App` in function `services.app_dsl_service.AppDslService.export_dsl` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:868:48
+ERROR Argument `SimpleNamespace` is not assignable to parameter `app_model` with type `App` in function `services.app_dsl_service.AppDslService.export_dsl` [bad-argument-type]
+   --> tests/test_containers_integration_tests/services/test_app_dsl_service.py:885:48

@dosu how to fix this.

[dosubot]

These type errors occur because SimpleNamespace is being passed where the type checker expects a real App instance. The fix is to replace SimpleNamespace(...) with actual App model instances.

Replace patterns like this:

app_model = SimpleNamespace(id="app_id", tenant_id="tenant_id", mode="chat")

With real App instances:

from models.model import App

app_model = App(
    tenant_id="test-tenant-id",
    name="Test App",
    mode="chat",  # or AppMode enum value
    enable_site=True,
    enable_api=True,
)

The id field is auto-generated via uuid4(), so you don't need to set it manually [1].

For the read-only property error on line 772 (app_model_config), if you need to set that attribute, you'll likely need to either:

• Create a real AppModelConfig object and associate it through the database relationship, or
• Use Mock(spec=App) which allows setting attributes freely

Since this is a test_containers_integration_tests file (with real DB access), the project's preferred approach is to use real ORM instances persisted to the test database rather than lightweight stand-ins [2] [3]. This gives you better type safety and more realistic test coverage.

Note: The current file on the main branch appears to be only ~435 lines, so the line numbers you're seeing (up to 885) may be from a branch or PR that significantly expanded the file. The same fix approach applies regardless.

To reply, just mention @dosu.

---

Share context across your team and agents. Try Dosu.

[asukaminato0721]

@dosu give me a list of file to update

[dosubot]

Based on the error lines, the file to update is:

api/tests/unit_tests/services/test_app_dsl_service.py [1]

(Note: despite the error message referencing test_containers_integration_tests, the file with lines up to 885 is the unit_tests version — the testcontainers version is only ~435 lines.)

Here's a breakdown of the SimpleNamespace usages that need replacing, grouped by the service method they're passed to:

Lines	Method Called	Parameter
138	internal import logic	existing_app → App
579, 607	AppDslService.check_dependencies	app_model → App
664, 738, 755, 793	AppDslService._create_or_update_app	app → App | None
772	property assignment	app_model_config (read-only)
833, 847	AppDslService.export_dsl	app_model → App
868, 885	AppDslService.export_dsl	app_model → App

Each SimpleNamespace(...) should be replaced with either a real App instance or Mock(spec=App) with the necessary attributes set. The App model [2] requires at minimum: tenant_id, name, mode, enable_site, and enable_api.

It's also worth noting that this is a broader pattern — there are ~16 test files across the test suite with similar SimpleNamespace usage that could trigger the same type errors [3] [4] [5] [6]. Let me know if you'd like the full list of those files as well.

To reply, just mention @dosu.

---

Docs are dead. Just use Dosu.

[asukaminato0721]

@dosu give me a file list across the whole project

[dosubot]

Here's the full list of files across the project that use SimpleNamespace where typed model instances are expected. I've organized them by directory and risk level.

---

🔴 High Impact (direct type mismatches with typed functions)

unit_tests/core/app/features/

• test_annotation_reply.py — App, Message (~10 type errors) [1]

unit_tests/services/workflow/

• test_workflow_human_input_delivery.py — App, Account (~8 type errors) [2]

unit_tests/services/

• test_app_dsl_service.py — App (~12 type errors) [3]
• test_dataset_service_dataset.py — Account, Dataset
• test_webhook_service.py
• test_async_workflow_service.py
• test_model_load_balancing_service.py
• test_annotation_service.py
• test_recommended_app_service.py
• test_code_based_extension_service.py
• test_workflow_run_service_pause.py
• workflow/test_workflow_event_snapshot_service.py
• workflow/test_workflow_restore.py

test_containers_integration_tests/controllers/web/

• test_conversation.py — App, EndUser, Conversation (~12+ type errors) [4]
• test_wraps.py — nested SimpleNamespace for feature flags (~14 instances) [5]
• conftest.py — factory functions make_app_model(), make_end_user() (high reuse impact) [6]

test_containers_integration_tests/services/

• test_file_service_zip_and_lookup.py — UploadFile [7]
• test_app_service.py — App
• test_workflow_app_service.py
• test_feedback_service.py — Account, Feedback
• test_human_input_delivery_test_service.py

test_containers_integration_tests/controllers/service_api/dataset/

• test_dataset.py — Tag [8]

integration_tests/controllers/console/app/

• test_feedback_export_api.py — Message, Conversation, App [9]
• test_chat_message_permissions.py — Conversation, Message [10]

---

🟡 Medium Impact

test_containers_integration_tests/controllers/web/

• test_web_forgot_password.py

test_containers_integration_tests/controllers/console/auth/

• test_email_register.py
• test_forgot_password.py

test_containers_integration_tests/controllers/console/workspace/

• test_workspace_wraps.py

unit_tests/core/app/task_pipeline/

• test_easy_ui_based_generate_task_pipeline_core.py
• test_based_generate_task_pipeline.py

unit_tests/core/app/apps/workflow/

• test_generate_task_pipeline_core.py
• test_app_config_manager.py
• test_app_generator_extra.py

unit_tests/core/app/layers/

• test_trigger_post_layer.py

unit_tests/core/workflow/nodes/human_input/

• test_human_input_form_filled_event.py
• test_entities.py

unit_tests/core/workflow/nodes/tool/

• test_tool_node_runtime.py
• test_tool_node.py

unit_tests/core/workflow/nodes/agent/

• test_runtime_support.py

unit_tests/core/workflow/

• test_node_runtime.py, test_node_factory.py, test_workflow_entry.py, test_workflow_entry_helpers.py, test_workflow_converter_additional.py, test_human_input_forms.py, test_human_input_compat.py, test_system_variable.py
• graph_engine/layers/test_llm_quota.py

unit_tests/core/app/

• features/test_hosting_moderation.py
• workflow/test_file_runtime.py, workflow/test_persistence_layer.py, workflow/test_observability_layer_extra.py, workflow/test_node_factory.py
• apps/test_pause_resume.py

unit_tests/core/prompt/

• test_simple_prompt_transform.py [11]
• test_prompt_transform.py

unit_tests/core/tools/

• workflow_as_tool/test_tool.py [12]
• workflow_as_tool/test_provider.py
• test_tool_manager.py, test_tool_label_manager.py, test_custom_tool_provider.py, test_custom_tool.py, test_tool_engine.py, test_builtin_tools_extra.py, test_dataset_retriever_tool.py, test_tool_file_manager.py
• utils/test_misc_utils_extra.py

unit_tests/core/plugin/

• test_model_runtime_adapter.py, test_backwards_invocation_app.py, test_backwards_invocation_model.py
• impl/test_debugging_client.py, impl/test_datasource_manager.py, impl/test_oauth_handler.py, impl/test_agent_client.py, impl/test_model_client.py, impl/test_tool_manager.py, impl/test_trigger_client.py

unit_tests/tasks/

• test_dataset_indexing_task.py — Dataset, Document (~45 usages)
• test_mail_human_input_delivery_task.py
• test_delete_account_task.py
• test_workflow_execute_task.py, test_human_input_timeout_tasks.py

unit_tests/repositories/

• test_sqlalchemy_workflow_node_execution_repository.py
• test_human_input_repository.py
• test_human_input_form_repository_impl.py

unit_tests/controllers/web/ (~15 files)

• test_audio.py, test_workflow_events.py, test_workflow.py, test_passport.py, test_site.py, test_app.py, test_web_passport.py, test_files.py, test_human_input_form.py, test_completion.py, test_message_endpoints.py, test_web_login.py, conftest.py, test_message_list.py, test_saved_message.py, test_remote_files.py

unit_tests/controllers/service_api/app/ (~8 files)

• test_audio.py, test_workflow.py, test_conversation.py, test_message.py, test_completion.py, test_workflow_fields.py, test_annotation.py

unit_tests/controllers/console/ (~7 files)

• app/test_wraps.py, app/test_audio_api.py, app/test_model_config_api.py, app/test_conversation_read_timestamp.py, test_workspace_account.py, test_workspace_members.py, test_init_validate.py, test_remote_files.py, workspace/test_tool_providers.py, workspace/test_load_balancing_config.py

unit_tests/libs/

• test_login.py [13]

unit_tests/misc/

• fields/test_file_fields.py, controllers/common/test_fields.py
• core/agent/output_parser/test_cot_output_parser.py
• core/ops/tencent_trace/test_client.py
• core/rag/rerank/test_reranker.py
• core/test_provider_manager.py

---

🟢 Lower Risk (config/flag mocking, ~57 RAG vector DB files)

unit_tests/core/rag/ — ~57 files for vector DB, extractor, indexing, and retrieval tests. These primarily mock configuration objects rather than SQLAlchemy models [14] [15].

---

That's roughly 166+ files total. The recommended fix is to replace SimpleNamespace with Mock(spec=ModelClass) or real model instances — the high-impact files listed at the top would be the best place to start.

To reply, just mention @dosu.

---

Share context across your team and agents. Try Dosu.

[iamPulakesh]

Hi, @asukaminato0721

I was working on something in the billing service (#34615) which I just finished submitting and I'd love to work on this one now. Could you assign this to me? I'll get started right away.