Move basic types from app to core (#10925)

We have a lot of low level types (e.g. FabricIndex) defined in
app/util/basic-types.h. These need to be used by other modules outside
of app (e.g. messaging, access control) which are lower level than app.

Right now the best low level place for these types is in lib/core. So,
moving them there for now, in a header called DataModelTypes.h. If we
need a more intermediate level (above core, below the other modules) in
the future, we can refactor then.

Issue #10924

src/app/util/basic-types.h

@@ -23,32 +23,18 @@
 
 #pragma once
 
-#include <stdint.h>
+#include <cstdint>
 
 #include <lib/core/GroupId.h>
 #include <lib/core/NodeId.h>
 
-// Pull in VendorId
+// Pull in other core types
 #include <lib/core/CHIPVendorIdentifiers.hpp>
+#include <lib/core/DataModelTypes.h>
 
 namespace chip {
-typedef uint8_t ActionId;
-typedef uint32_t AttributeId;
-typedef uint32_t ClusterId;
-typedef uint8_t ClusterStatus;
-typedef uint32_t CommandId;
-typedef uint32_t DataVersion;
-typedef uint32_t DeviceTypeId;
-typedef uint16_t EndpointId;
-typedef uint32_t EventId;
-typedef uint64_t EventNumber;
-typedef uint64_t FabricId;
-typedef uint8_t FabricIndex;
-typedef uint32_t FieldId;
-typedef uint16_t ListIndex;
-typedef uint32_t TransactionId;
+
 typedef uint8_t Percent;
 typedef uint16_t Percent100ths;
 
-static constexpr FabricIndex kUndefinedFabricIndex = 0;
 } // namespace chip

src/lib/core/BUILD.gn

@@ -101,6 +101,7 @@ static_library("core") {
     "CHIPTLVUpdater.cpp",
     "CHIPTLVUtilities.cpp",
     "CHIPTLVWriter.cpp",
+    "DataModelTypes.h",
     "GroupId.h",
     "NodeId.h",
     "PeerId.h",

src/lib/core/DataModelTypes.h

@@ -0,0 +1,42 @@
+/*
+ *
+ *    Copyright (c) 2021 Project CHIP Authors
+ *
+ *    Licensed under the Apache License, Version 2.0 (the "License");
+ *    you may not use this file except in compliance with the License.
+ *    You may obtain a copy of the License at
+ *
+ *        http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *    Unless required by applicable law or agreed to in writing, software
+ *    distributed under the License is distributed on an "AS IS" BASIS,
+ *    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *    See the License for the specific language governing permissions and
+ *    limitations under the License.
+ */
+
+#pragma once
+
+#include <cstdint>
+
+namespace chip {
+
+typedef uint8_t ActionId;
+typedef uint32_t AttributeId;
+typedef uint32_t ClusterId;
+typedef uint8_t ClusterStatus;
+typedef uint32_t CommandId;
+typedef uint32_t DataVersion;
+typedef uint32_t DeviceTypeId;
+typedef uint16_t EndpointId;
+typedef uint32_t EventId;
+typedef uint64_t EventNumber;
+typedef uint64_t FabricId;
+typedef uint8_t FabricIndex;
+typedef uint32_t FieldId;
+typedef uint16_t ListIndex;
+typedef uint32_t TransactionId;
+
+static constexpr FabricIndex kUndefinedFabricIndex = 0;
+
+} // namespace chip