generatedunixname226714639793621

This commit performs a **refactoring** within the **Thrift compiler test fixtures** to explicitly add field IDs to method parameters. Specifically, it adds a field ID to the `input` parameter of the `echo` service method in `thrift/compiler/test/fixtures/py-cpp-transport/src/module.thrift`. Additionally, a field ID is added to the `num` parameter of the `isPrime` service method in `thrift/compiler/test/fixtures/py-future/src/test.thrift`. This change enhances the robustness and consistency of these internal test definitions, with no direct impact on production Thrift services or user-facing APIs.

This commit performs a **refactoring** of **Thrift IDL definitions** by explicitly adding field IDs to parameters in several test services. Specifically, it assigns a field ID to the `initialSum` parameter of the `basicInteractionFactoryFunction` within the `RPCConformanceService` defined in `thrift/conformance/if/rpc.thrift`. Additionally, field IDs are introduced for parameters in the `echo` and `overflow` methods of the `Sleep` service, located in `thrift/lib/py/server/test/sleep.thrift`. This ensures proper schema definition and compatibility for these **Thrift test services**, aligning them with best practices for robust serialization and deserialization.

This commit **refactors** the **FBOSS agent's `TestCtrl` Thrift service** by adding explicit field IDs to boolean parameters in the `setSelfHealingLagState` and `setConditionalEntropyRehash` methods. This **maintenance** change, applied to `fboss/agent/if/test_ctrl.thrift`, improves the robustness and forward compatibility of the Thrift interface definition. By ensuring explicit identification of parameters, it enhances the **stability of the testing control plane** and prevents potential issues with future interface modifications. This update impacts how clients interact with these specific test control functions, requiring recompilation of Thrift stubs.

This commit performs a **refactoring** by systematically adding the `@thrift.AllowUnsafeRequiredFieldQualifier` annotation to numerous **required fields** within various Thrift IDL files. This annotation is applied to structs and exceptions across test and example schemas for the **C++, Java, Python, and Rust Thrift libraries**. The change prepares these definitions for potential future compiler optimizations or relaxed validation rules concerning the presence of `required` fields. This widespread update ensures consistency and enables a new mechanism for managing the strictness of `required` field handling within the broader **Thrift ecosystem**.

This commit performs a **refactoring** of the **Thrift definitions** for core network address types. It adds the `@thrift.AllowUnsafeRequiredFieldQualifier` annotation to required fields within the `Address`, `BinaryAddress`, and `IPPrefix` structs defined in `common/network/if/Address.thrift`. This change specifically addresses **Thrift schema management** by explicitly allowing required fields to be qualified, which can impact **Thrift code generation** and ensure compatibility or suppress warnings in downstream services consuming these core networking data structures.

This commit description cannot be generated as the necessary input, including a specific commit message, work classification, and detailed file-level changes with semantic information, was not provided. To produce an accurate summary of the **code modifications**, affected **modules**, and the **nature of the work**, comprehensive details about the changes are required. Please provide the actual commit message and a list of file changes with their semantic context for a proper description.

This commit **refactors** the `thrift/compiler/test/const_util_test.thrift` file by adding an include for `thrift/annotation/thrift.thrift`. It then applies the `AllowLegacyMissingUris` annotation to the package declaration within this **Thrift compiler test**. This **maintenance** change ensures the test suite correctly handles or explicitly allows legacy behavior regarding missing URIs, preventing potential issues with future compiler updates or stricter validation rules. The modification is confined to the **Thrift compiler's test infrastructure**, ensuring its continued stability and compatibility.

This commit performs a **maintenance update** across a broad set of **Thrift test files** by adding the `thrift.AllowLegacyMissingUris` annotation and its include to their package declarations. This change affects numerous test schemas within the `reflection`, `rust`, `tablebased`, and `thrift-python` modules, among others. The work is a **test infrastructure adjustment** designed to ensure that existing test cases remain compatible with new Thrift compiler behavior or requirements concerning URI handling. Its primary impact is to prevent potential compilation errors or warnings in the test suite by explicitly allowing for legacy scenarios where URIs might be absent.

This commit **updates numerous Thrift test files** by adding the `@thrift.AllowLegacyMissingUris` annotation and a package declaration to each. This **maintenance task** ensures that the existing **Thrift testing infrastructure** continues to function correctly by explicitly allowing legacy URI resolution behavior for these test definitions. The change is a **preparatory step** to maintain compatibility and prevent potential breakage in tests as Thrift's URI handling evolves, impacting a wide range of test cases within the `thrift/test/` directory.

This commit performs a **maintenance chore** across a large number of **Thrift test definition files** within the `thrift/test/` directory. It systematically adds the `@thrift.AllowLegacyMissingUris` annotation and a package declaration to each affected `.thrift` file. This **refactoring** effort ensures the **Thrift test suite** remains compatible with evolving Thrift language requirements, specifically concerning URI resolution and module organization. By explicitly allowing legacy URI behavior, this change prevents potential future compilation or runtime issues in the tests, aligning them with current best practices for Thrift definitions.

This commit performs a **maintenance update** to several **Thrift test definition files** within `thrift/lib/cpp2`. It adds the `@thrift.legacy_missing_uris` annotation and a `package` declaration to test utility files such as `thrift/lib/cpp2/transport/rocket/test/util/ThriftService.thrift` and `thrift/lib/cpp2/util/test/Matcher.thrift`. This ensures **compatibility** for test cases that might involve older Thrift definitions or specific URI handling, allowing them to properly generate and function. The changes are confined to **internal testing infrastructure**, with no impact on the core `thrift/lib/cpp2` runtime or public APIs.

This commit performs a **refactoring** of the **Thrift performance testing interfaces** by introducing a package declaration and a specific annotation for legacy missing URIs. The `ApiBase.thrift` and `load.thrift` definitions, located in the `thrift/perf` directory, are updated with these structural improvements. This change aims to enhance the **standardization and compatibility** of these Thrift definitions, particularly in how legacy URI handling is declared. Ultimately, this update contributes to better **maintainability and clarity** within the Thrift performance framework.

This commit performs a **maintenance update** across a wide range of **Thrift definition files** within the `thrift/lib/java`, `thrift/lib/javadeprecated`, `thrift/lib/py`, and `thrift/lib/py3` directories. It **adds a specific Thrift annotation** to each of these files, which is designed to **allow for legacy missing URIs and package declarations**. This change is a **compatibility improvement**, ensuring that older or less explicitly defined Thrift schemas can be processed correctly by the Thrift compiler and runtime. The broad application of this annotation across numerous test and example `.thrift` files indicates a significant effort to **streamline migration** and maintain backward compatibility for existing Thrift services and data structures.

This commit performs a **maintenance** update across numerous **FBOSS platform services** and the **MKA service**. It consistently adds **Thrift annotations and package declarations** to various `.thrift` definition files, including those for `mka_service`, `bsp_tests`, `data_corral_service`, `fan_service`, `platform_manager`, `rackmon`, and `sensor_service`. This **standardization of Thrift schemas** improves consistency and prepares these interfaces for better tooling integration or future language bindings. The changes are purely definitional and do not alter the functional behavior of the affected services.

This commit **refactors** numerous **Thrift definition files** across the FBOSS codebase by adding specific annotations. These annotations enable **legacy URI allowance** and include **package declarations** within the Thrift schemas. This change affects various subsystems such as `fb303`, `network`, `fsdb`, `led_service`, and several `lib` components, standardizing their Thrift definitions. The primary goal is to improve compatibility and manage Thrift schema evolution by explicitly handling legacy URI resolution and formalizing package structures, ensuring smoother integration and future development.

This commit performs a **refactoring** of the **Thrift configuration definitions** for both **SDK Castle** and **LLDP**. It introduces new Thrift annotations to `fboss/util/sdkcastle/sdkcastle_config.thrift` and `fboss/util/thrift/LldpConfig.thrift`, specifically to allow for legacy missing URIs. Additionally, a package declaration is added to these Thrift files, improving **code organization** and **compatibility** with older systems. This change enhances the robustness of the configuration parsing by accommodating historical data formats.

This commit performs a significant **refactoring** across numerous **Thrift definition files** within the `fboss/agent` and `fboss/agent/hw` components. It systematically adds `thrift.uri` annotations to address legacy missing URIs and introduces `package` declarations to files such as `agent_config.thrift`, `bcm_config.thrift`, and various interface definitions under `fboss/agent/if/`. This **maintenance** effort improves the **metadata and organization** of the Thrift schemas, enhancing their compatibility and potentially streamlining code generation or cross-service communication for the agent's core interfaces and hardware abstractions. The broad scope of this update ensures consistent Thrift schema practices across a wide array of FBOSS agent modules.

This commit performs a **maintenance update** to several **FBOSS agent Thrift interface definitions** and associated test files. It **adds `AllowLegacyMissingUris` annotations and package declarations** to core schema files like `product_info.thrift`, `sflow.thrift`, `lldp.thrift`, and various `switch_config` related Thrift definitions. This **chore** ensures compatibility with evolving Thrift tooling and practices, specifically by allowing older, potentially unversioned, Thrift URIs to be handled gracefully. The changes are primarily to the **schema definitions** themselves, impacting how these interfaces are generated and consumed across the FBOSS system without altering functional behavior.

This commit performs a significant **refactoring** across numerous **FBOSS Thrift interface definitions** by adding essential Thrift annotations for legacy URI handling and explicit package declarations. This standardization effort impacts a broad range of subsystems, including `fboss/qsfp_service`, `fboss/platform/weutil`, `fboss/thrift_cow`, `fboss/thrift_visitors`, and `fboss/util/sdk_onboarding`. The work aims to ensure consistent Thrift schema management, which is crucial for facilitating future migrations and improving compatibility for services consuming these interfaces. These changes are purely structural and do not alter the functional behavior of the existing APIs, serving as a foundational step for broader Thrift infrastructure updates.

This commit **refactors** the annotation usage within the **Thrift test fixtures** for the Python interface (pyi) generation. It updates the `thrift/compiler/test/fixtures/pyi/src/shared.thrift` file to apply the `@thrift.DeprecatedUnvalidatedAnnotations` annotation to `SimpleStruct` and its fields `a_str` and `a_bytes`. This **refactoring** ensures that the test definitions align with updated annotation validation mechanisms within the **Thrift compiler**, improving consistency and maintainability of the test suite. The change primarily affects the **Thrift definition files** used for testing the compiler's pyi output.